-
Notifications
You must be signed in to change notification settings - Fork 21
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
Ability to control the product selection of synced products #2006
Conversation
Provides the ability to control the product selection of synced products.
Add apply_filters hook to prepare_query_args
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #2006 +/- ##
=========================================
Coverage 57.9% 57.9%
Complexity 4111 4111
=========================================
Files 456 456
Lines 17658 17659 +1
=========================================
+ Hits 10224 10225 +1
Misses 7434 7434
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Hi @andyg2
Thanks for opening this PR. Only two minor things for getting approval:
- Changing the name of the hook following the name convention of the other hooks (see my comment)
- Fix the PHPCS errors
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.
✅ LGTM - thanks for your contribution @andyg2
FYI - This will be released potentially next Tuesday 25th July. |
Provides the ability to control the product selection of synced products.
Changes proposed in this Pull Request:
Adding an apply filters hook allows users more control over which products can be selected for syncing.
Closes #2005
Additional details:
Tweak - Ability to filter products for syncing via
gla_filter_product_query_args
apply_filters hook.Changelog entry
Tweak - Ability to filter products for syncing via
gla_filter_product_query_args
apply_filters hook.