Skip to content

Commit 7731130

Browse files
committed
Merge pull request twitter#16 from augbog/master
Added fix so pulls total number of members
2 parents a43c968 + edd7cf7 commit 7731130

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
@@ -137,13 +137,26 @@
137137
}
138138
addRepos();
139139

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

148161
function randomItem(array) {
149162
return array[Math.floor(Math.random() * array.length)];
@@ -230,7 +243,6 @@
230243
}
231244
});
232245
}
233-
234246
})(jQuery);
235247
</script>
236248
</head>

0 commit comments

Comments
 (0)