Skip to content
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

add woocommerce related product widget #850

Conversation

ivankristianto
Copy link
Contributor

@ivankristianto ivankristianto commented May 28, 2017

Changelogs:

  1. Add new widget: EP_Related_Products_Widget
  2. Add new function: ep_wc_get_related_products

Fixed #736

@ivankristianto ivankristianto requested a review from tlovett1 May 28, 2017 08:50
@tlovett1 tlovett1 added this to the 2.4 milestone May 30, 2017
@tlovett1 tlovett1 removed this from the 2.4 milestone Sep 26, 2017
@tlovett1
Copy link
Member

tlovett1 commented Oct 3, 2017

I'm experiencing some strange behavior:

I have three products with similar names. When looking at any of the products, they all show a related products which makes sense. When I add a product tag to any of them, none show in the widget.

What is the expected functionality for this? How does the algorithm work?

@ivankristianto
Copy link
Contributor Author

The way it works is using ES more_like_this query:
https://www.elastic.co/guide/en/elasticsearch/reference/5.5/query-dsl-mlt-query.html

So here is the query in Related product widget:
https://github.com/10up/ElasticPress/blob/develop/features/related-posts/related-posts.php#L10

Where the more_like query is introduce in:
https://github.com/10up/ElasticPress/blob/develop/features/related-posts/related-posts.php#L10
Which I think we should move this to the ep_formatted_args to make it usable in WP_Query without activate Related posts feature.

@allan23
Copy link
Contributor

allan23 commented Feb 20, 2019

Thank you for your interest in contributing to ElasticPress. At the time of this writing, we are focusing our development efforts on version 3.0 in the develop branch, which is an entirely different codebase. We ask that you please take some time to review the new codebase and alter your pull request as needed to accommodate these major changes.

@allan23 allan23 closed this Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants