-
Notifications
You must be signed in to change notification settings - Fork 643
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
More support for keyset pagination #1144
Comments
Also allow order_by to be set in globally in config Fixes python-gitlab#1144
Also allow order_by to be set in globally in config Fixes python-gitlab#1144
Also allow order_by to be set in globally in config Fixes python-gitlab#1144
As mentioned before:
Keeping it open for now and let's see what GitLab does. I don't think we should do anything right now or break peoples workflow |
So x-total header is not returned on gitlab.com when results have more than 10k results: https://docs.gitlab.com/ee/user/gitlab_com/index.html#pagination-response-headers This makes things break in strange ways for code which works for some time, but then results grow to more than 10k and it stops working. I think this library should simply not encourage use of x-total pagination at all, or at least make it so that you have to opt-in if you really know what you are doing. @ericfrederich See also #961. |
Description of the problem, including code/CLI snippet
Initial keyset pagination support was added in commit 0b71ba4, however it is still not:
Expected Behavior
Pagination should be able to be set in the following places in order of precedence:
Actual Behavior
Pagination can only be set in config file under a particular gitlab id.
Specifications
master
branchv4
13.2.3
The text was updated successfully, but these errors were encountered: