Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update facebook.py #47

Closed
wants to merge 1 commit into from
Closed

Update facebook.py #47

wants to merge 1 commit into from

Conversation

allfro
Copy link

@allfro allfro commented Sep 15, 2012

Added iterating methods for paging

Added iterating methods for paging
@martey
Copy link
Member

martey commented Sep 21, 2012

I think my comments on pull request #37 still apply - these new next and previous methods are of limited utility, and there is unnecessary duplication from the existing request method.

Since Facebook looks to be deprecating previous-next pagination, I think this is not the right time to implement something like this.

@martey martey closed this Sep 21, 2012
@rbpasker
Copy link

rbpasker commented May 4, 2013

martey -- you've closed two pull requests with pagination solutions. could you please add something to docs that explains exactly how you expect people using the API to do paging?

( #37 )

@vmassuchetto
Copy link

+1 for documenting the paging functionality. It's confusing how to do this with this API.

@skyler
Copy link

skyler commented Jan 3, 2014

I'm sorry if I've missed this elsewhere, but how should we do pagination using this API? Should I make an HTTP request using the URL returned in next and previous?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants