-
-
Notifications
You must be signed in to change notification settings - Fork 848
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
Enable the pagination via cursor (no page-based pagination) #2532
Conversation
215c084
to
dee63df
Compare
be4b2c0
to
832cc14
Compare
Doc pr : api-platform/docs#738 |
Add tests for the SoMany collection Uses the PropertyAccessor service and don't break the BC Allow to use multiple fields Fix cs Improve cursor-based pagination tests
832cc14
to
472609c
Compare
@soyuka We might still be missing something here that we were probably not doing back when the initial PR was open. I think we should add a |
Why this was not released in 2.4 ? 🙂 |
Because it did not land in 2.4 but in master, it's an experimental feature. |
Forgot about that lower branches are merged to master.. |
Add tests for the SoMany collection
Uses the PropertyAccessor service and don't break the BC
Allow to use multiple fields
Fix cs
To avoid breaking @sroze branch I'm creating a new PR with his work. Original PR is #1915 . I only rebased, squashed and fixed CS.