-
Notifications
You must be signed in to change notification settings - Fork 408
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
Documentation: Check the consistency with page number assignation change in 9+ #739
Comments
incase others need this and this is closed as 'not fixing' i monkey patched it by
|
@phyzical thank you for your report. I am wondering how you don't get the automatic assignation of the Could you please use one of the Playground apps to show how to make it fail? |
I assume then it is just simply the fact its a redundant spec causing the issue to begin with, as its explicitly passing in the param. Based on what your saying its probably safe to just close this. Just figured id raise it as it does seem to be a functionality change not mentioned in the breaking changes.
|
and why it happens
|
That explains it. Indeed, if you explicitly pass a variable, you are responsible for passing it right.
I will check whether the docs and breaking changes are consistent with the current behavior before closing it. |
👀 Before submitting...
🧐 REQUIREMENTS
💬 Description
Not sure if this is a bug but in 9 we had a set of tests fail that assumed that invalid page numbers result in the first page to be returned i.e providing
page: -1
page: 'blah'
in 8 would result in the first page of results
in 9 we get
TBH, i think its just a set of silly tests ourside but i could see it biting larger projects where people do dodge to reset pages ect.
Might just be worth getting added to the breaking changes list?
The text was updated successfully, but these errors were encountered: