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

Pagination Links on AMS #1040

Closed
bacarini opened this issue Aug 5, 2015 · 2 comments
Closed

Pagination Links on AMS #1040

bacarini opened this issue Aug 5, 2015 · 2 comments

Comments

@bacarini
Copy link
Contributor

bacarini commented Aug 5, 2015

Hey guys,

I am working on an API and I need to return pagination links on my responses. I am wordering the best way to to that. JSON-API specification does not let it clear on how it should display the links top-level.

what is your thoughts about the following example?

{
    "data": [..],
    "links": {
        "pages": {
            "first": "?page=1&per_page=1",
            "prev": "?page=2&per_page=1",
            "next": "?page=4&per_page=1",
            "last": "?page=10&per_page=1"
        }
    }
}

Anyway, I've opened a PR and I would like to discuss it with you all.

@bacarini
Copy link
Contributor Author

@joaomdmoura,

I think we can close this issue, since #1041 was merged, no?

@joaomdmoura
Copy link
Member

Indeed, closed! 😄

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

No branches or pull requests

2 participants