-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
28628 price range wildcards #28745
28628 price range wildcards #28745
Conversation
- Removed wildcard usage for price filters
…price_range_wildcards
- Fixed price range wildcard values and associated test cases
…price_range_wildcards
Hi @gallyamov. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. |
@magento run all tests |
- Code style fixes
- Fixed tests
@magento run all tests |
@gallyamov please make sure api-functional tests are updated to reflect this change, and that this change is covered by api-functional tests for GraphQl perspective. There are already some tests that should be failing now with this change, so they should be updated with new behavior |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just need to update api-functional tests and make all builds pass
unset($value['from'], $value['to']); | ||
|
||
$rangeName = "{$from}_{$to}"; | ||
$rangeName = "{$value['from']}_{$value['to']}"; | ||
$resultData[$rangeName] = array_merge(['value' => $rangeName], $value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is causing static failure due to array_merge in loop. can it be refactored to avoid that?
…price_range_wildcards
- Fixed tests
@magento run all tests |
Hi @gallyamov, thank you for your contribution! |
Description (*)
This is for #28628
Related Pull Requests
N/A
Fixed Issues (if relevant)
Manual testing scenarios (*)
Check fixed issue for testing steps and expected results.
Questions or comments
N/A
Contribution checklist (*)