Skip to content

Correct paging with iterator #75

Merged
merged 2 commits into from Jul 23, 2012

1 participant

@kevinburke

The next_page_uri includes filters, so you don't need to continue adding them
to subsequent API requests. If the next_page_uri is being used, just pass an
empty array for the filters.

Also add a test for the correct behavior in this case, based on the real API
response.

Kevin Burke added some commits Jul 23, 2012
Kevin Burke Merge test from upstream ab26255
Kevin Burke Remove filters from paging with next_page_uri
The next_page_uri includes filters, so you don't need to continue adding them
to subsequent API requests. If the next_page_uri is being used, just pass an
empty array for the filters.

Also add a test for the correct behavior in this case, based on the real API
response.
f40f331
@kevinburke kevinburke merged commit f40f331 into master Jul 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.