Skip to content

Commit a81ae1b

Browse files
author
Mike Taylor
committed
Issue #372 - Add default per_page and update page regex.
1 parent d4cbd9c commit a81ae1b

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

webcompat/static/js/lib/models/issue.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,8 @@ issueList.Issue = issues.Issue.extend({});
120120

121121
issueList.IssueCollection = Backbone.Collection.extend({
122122
model: issueList.Issue,
123-
url: '/api/issues?page=1',
123+
// TODO(miket): less hard-coding of default params
124+
url: '/api/issues?page=1&per_page=50',
124125
parse: function(response, jqXHR) {
125126
if (jqXHR.xhr.getResponseHeader('Link') != null) {
126127
//external code can access the parsed header via this.linkHeader
@@ -168,7 +169,7 @@ issueList.IssueCollection = Backbone.Collection.extend({
168169
}
169170
var page;
170171
// we only return the page number
171-
var re = new RegExp('page=(\\d)');
172+
var re = new RegExp('[&?]page=(\\d)');
172173

173174
if (page = (this.linkHeader.hasOwnProperty(relation) &&
174175
this.linkHeader[relation].match(re))) {

0 commit comments

Comments
 (0)