Enhance CollectionService to support cursor-based pagination #1355
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…th beforeEq and afterEq parameters. Added error handling for invalid cursor requests and improved item retrieval logic. Updated response structure to include cursor URIs for navigation.
…correct id to the response.
- Modify collection service to support flexible pagination - Remove totalItems from collection tests - Update pagination test cases to use cursor-based navigation - Adjust collection retrieval and sorting logic - Add more flexible item retrieval conditions
- Expand collection test suite with comprehensive pagination scenarios - Add edge case and error handling tests for collection pagination - Improve test structure with more granular test groups
…takenly removed in a previous commit
…elude to refactoring the get method for a better readability and maintainability. - Extract collection get logic to a separate file (collection-get.js) - Import getAction from new external file - Add getValueFromDataType to methods - Simplify collection service structure
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #568