feat(pagination): Add UuidRangeFilter to allow Uuid cursors#4689
feat(pagination): Add UuidRangeFilter to allow Uuid cursors#4689KDederichs wants to merge 13 commits intoapi-platform:mainfrom
Conversation
|
Where is the status on this PR ? :) |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This is still really needed. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
this is still needed |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This is still needed |
|
What’s preventing this from being merged? |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This is still needed. |
|
@soyuka With the emergence and popularity of UUIDv7 this become rather relevant, at least if you try to do cursor based pagination with that. |
|
#5618 has reasons why this still hasn't been merged... |
# Conflicts: # .github/workflows/ci.yml # src/Doctrine/Common/Filter/RangeFilterTrait.php # src/Doctrine/Odm/Filter/RangeFilter.php # src/Doctrine/Orm/Filter/RangeFilter.php # tests/Fixtures/app/config/config_test.yml
|
I've reviewed this a bit and I really like the work, I'd like try to find some time to refactor/merge this. |
start for api-platform#4689
As requested in #4595 I moved the Uuid Range to be its own thing.
I hope I managed to catch all the services that need to be wired if not let me know!