PR#462: Fix pagination metadata to include query params and next/prev pages -

This fixes:
1. Pagination does not include query parameters
2. 'prev' page is missing for first page and 'next' page is missing
for last page.

Optional: Since the tests use Flask >= 1.0, you can use resp.json instead of loading it yourself.

Could you also check that the query parameter is in meta['next']? That's actually what I was using when I noticed the bug.

Same as above

rebased onto 88c14a6d8470638d6ec0cc27561a8b0d428cea1f

Jan 22, 2020

@mprahl patch is updated per comments, thanks.

Could you please be consistent with the usage of quotes vs double quotes within a method?

@qwan this looks good! Once the quotes are fixed, this is good to merge.

rebased onto 89602990dc659acdeab396e18b41b2c86a44805c

Jan 23, 2020

Fixed the inconsistent quotes, going to merge this.

Pull-Request has been merged by qwan

Jan 23, 2020

Assignee

None

Tags

No Tags