-
Notifications
You must be signed in to change notification settings - Fork 1
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
Pagination #7
Comments
I'll have to check with @elbrujohalcon how we usually do this, but I think it depends on the project. Sometimes we use page numbers, sometimes we use a cursor. As far as I know, the paginated resources are the only ones that should get enveloped items, as you wrote above @thiagogabriel . I'm gonna check if there's no item on the momentary index for this and, if there isn't, add it . |
Pagination can be in the headers, right @volbap ? |
In any case, we can totally add pagination as an exception to that rule, if needed. |
Yeah, I would use the link headers as explained here. It seems to be a fair solution. Also, let's try to avoid envelopes as well. |
What Jayme currently supports is a non-cursor based pagination, using these headers: This kind of pagination, however, may not be quite useful for certain common scenarios, like a scroll feed. So, I would not make it a standard. |
Since we should return a plain array with the items, where would we include the pagination metadata ?
The practice I know is do something like this:
The text was updated successfully, but these errors were encountered: