ElasticPress
ElasticPress copied to clipboard
Limit WooCommerce integration with ability to allow it
Description of the Change
Filter to allow WooCommerce. It will limit the usage of WooCommerce integration to the places where it is actually needed.
Closes #2809
How to test the Change
Changelog Entry
Added - Filter to allow/limit WooCommerce integration Changed -
should_integrate_with_query
function
Credits
Props @MARQAS
Checklist:
- [x] I agree to follow this project's Code of Conduct.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my change.
- [ ] All new and existing tests pass.
@MARQAS before we merge this one we will need a list of real scenarios where we would expect the feature to integrate or not with Elasticsearch. Then that will need to become proper tests. Can you please work on that? Thanks.
https://github.com/10up/ElasticPress/pull/2914/commits/f66b88011ac471fd8fa013025024fc641626b443 looks good but am not sure if the two instances of ep_integrate removed from tests is all we need. Did we create the list of scenarios @MARQAS ?
@MARQAS I've created a new sheet in our doc with a format I think will make things easier for us to understand. Can you please let me know when you have that completely filled? Feel free to add any new lines/scenarios you think I missed. After we have that complete, we can plan to write tests for each scenario. Thanks!
Closing this in favor of (the already merged) #3259