-
Notifications
You must be signed in to change notification settings - Fork 0
Add logical cursor for long query parameters #7
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
Open
JoMessina
wants to merge
25
commits into
main
Choose a base branch
from
feature/add-cursor-for-long-filter
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
5adecfa
Add logical cursor for long query parameters
JoMessina d32cc33
fix type phpstan, manage api return's cursor with the array of parame…
JoMessina 1cdb190
remove IGNORE_ENV_TRUE parameter for cs-fixer
JoMessina c03dc1d
Ignore a rule from phpstan
JoMessina 3c8721e
fix from cs fixer
JoMessina 9748ff3
Merge branch 'main' into feature/add-cursor-for-long-filter
JoMessina b0ca2a1
fix from cs fixer
JoMessina 3a5066c
change phpstan config for level 5
JoMessina 1442b14
Change a method name
JoMessina e973d99
Refactored the way filters are handled
gplanchat 730e90b
[rector] Rector fixes
actions-user c0a80ee
refacto extractors, add withGroups and withFilters methods
JoMessina afc78d5
manage page count, fix extractor to use api pagination
JoMessina 047ae6e
fix and add tests for extractors
JoMessina 8854a9c
Merge branch 'main' into feature/add-cursor-for-long-filter
JoMessina 7930389
[rector] Rector fixes
actions-user d9edd24
Refactored the way filters are handled
gplanchat 784d8a4
[rector] Rector fixes
actions-user 51dbb92
Fixed errors in the unit tests
gplanchat 0ce5f65
Changed the variadic methods to use array_push instead of foreach
gplanchat 912c39c
remove a cast on string, add a catch for UndefinedOptionsException
JoMessina 8f501aa
[rector] Rector fixes
actions-user dca47a4
Fix client api version to 2.4
JoMessina 9083255
update api-client-magento to lock symfony/serializer
JoMessina edfe529
valid phpstan 7, use rector and cs-fixer
JoMessina File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.