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

Provide a max_per_page parameter for pagination #384

Closed
dmulter opened this issue Mar 25, 2016 · 1 comment
Closed

Provide a max_per_page parameter for pagination #384

dmulter opened this issue Mar 25, 2016 · 1 comment
Milestone

Comments

@dmulter
Copy link

dmulter commented Mar 25, 2016

Right now the default for per_page is 20 unless otherwise specified. I also see that the code makes sure it is no larger than the total items.

To prevent a request asking for too many items at once, it would be convenient if Pagination could be initialized with a maximum items per page (max_per_page). Then the supplied per_page value could use the ceil() of total and max_per_page.

@kalombos
Copy link
Contributor

#542

@davidism davidism closed this as completed Oct 4, 2017
@davidism davidism added this to the 2.3.1 milestone Oct 4, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

3 participants