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

Refactor Collection Paging #199

Merged
merged 1 commit into from Aug 26, 2015

Conversation

Projects
None yet
4 participants
@jasnell
Copy link
Collaborator

commented Aug 12, 2015

Per:#193

This refactors the way paging works based on feedback from @azaroth42

The changes are fairly significant but result in a much cleaner
paging model that is more consistent with other paging efforts
(it's still distinct from LDP paging, for instance, but should be
far more compatible).

@jasnell jasnell referenced this pull request Aug 12, 2015

Closed

as:Collection semantics #193

@azaroth42

This comment has been minimized.

Copy link
Contributor

commented Aug 12, 2015

non binding 👍

@kevinmarks

This comment has been minimized.

Copy link

commented Aug 25, 2015

How does this new paging model work? that wasn't clear from the changes.
Twitter's model is interesting in the context of improving the api https://dev.twitter.com/rest/public/timelines

Refactor Collection Paging
Per:#193

This refactors the way paging works based on feedback from @azaroth42

The changes are fairly significant but result in a much cleaner
paging model that is more consistent with other paging efforts
(it's still distinct from LDP paging, for instance, but should be
far more compatible).

@jasnell jasnell force-pushed the refactor-paging branch from a6a88a9 to 22490a1 Aug 26, 2015

@jasnell

This comment has been minimized.

Copy link
Collaborator Author

commented Aug 26, 2015

Today the WG resolved to land this change.

jasnell added a commit that referenced this pull request Aug 26, 2015

@jasnell jasnell merged commit 62dbd70 into master Aug 26, 2015

@jasnell jasnell deleted the refactor-paging branch Oct 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.