Skip to content

Commit edd7cf7

Browse files
author
Augustus Yuan
committed
added fix so pulls total number of members
1 parent f15da8d commit edd7cf7

File tree

1 file changed

+19
-7
lines changed

1 file changed

+19
-7
lines changed

index.html

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,13 +136,26 @@
136136
}
137137
addRepos();
138138

139-
$.getJSON("https://api.github.com/orgs/twitter/members?callback=?", function (result) {
140-
var members = result.data;
141-
142-
$(function () {
143-
$("#num-members").text(members.length);
139+
//get total number of Twitter members
140+
function getNumMembers(page, numMembers) {
141+
var page = page || 1;
142+
var numMembers = numMembers || 0;
143+
var membersUri = "https://api.github.com/orgs/twitter/members?callback=?"
144+
+ "&per_page=100"
145+
+ "&page="+page;
146+
147+
$.getJSON(membersUri, function (result) {
148+
if (result.data && result.data.length > 0) {
149+
numMembers += result.data.length;
150+
getNumMembers(page+1, numMembers);
151+
} else {
152+
$(function () {
153+
$("#num-members").text(numMembers);
154+
});
155+
}
144156
});
145-
});
157+
}
158+
getNumMembers();
146159

147160
function randomItem(array) {
148161
return array[Math.floor(Math.random() * array.length)];
@@ -229,7 +242,6 @@
229242
}
230243
});
231244
}
232-
233245
})(jQuery);
234246
</script>
235247
</head>

0 commit comments

Comments
 (0)