-
Notifications
You must be signed in to change notification settings - Fork 19
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
Allow more than 100 posts per page in queries #228
Comments
Since the limit (of currently 100 posts) is stored in a public property, it can be adapted on a per-project basis. Do you mean we should do this in the If yes, do you think 2000 is a good new value? |
I took a quick look at the HM repos, total of 19 ignores with these values: 1 x 150 |
I did another search for anything beyond 1000. No custom code is using more than Thoughts? |
100 posts per page is a pretty low limit really, and in actuality we can allow probably another order of magnitude here. The 100 per page decision in the REST API that we made was on the basis that each could embed other items, so 100 would actually be many more than 100 queries.
In custom code though, that's not necessarily the case, and allowing more would not necessarily be a problem. As an example, the sitemaps in WP core could query up to 2000 posts at a time, and that's probably not going to be an issue; see humanmade/altis-seo#38 (comment)
Should we increase this limit?
The text was updated successfully, but these errors were encountered: