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

fallback to magento category product positions #2523

Open
wants to merge 1 commit into
base: 2.10.x
Choose a base branch
from

Conversation

Cloudy777
Copy link

uses magento default product category position if non are set in elasticsuite table to respect position of migrations or imports

uses magento default product category position if non are set in elasticsuite table to respect position of migrations or imports
@romainruaud romainruaud added the safe to test PR can be tested label Apr 14, 2022
@rbayet
Copy link
Collaborator

rbayet commented Sep 8, 2023

Hello @Cloudy777,

What about people who migrated from a non-Elasticsuite install with legacy product positions to an Elasticsuite install seeing the benefit of Optimizers for automated / semi-manual category merchandizing ?

At the first mandatory reindex, old positions would creep pop in the search engine index and mess with the natural ordering of products in a ninja way since those positions will not be displayed as "manual positions" in the category preview zone.

Any optimizer applicable to catalog navigation would seem to randomly work or not.

I'd be more in favor of a (documented) cli command to migrate the legacy positions into the Elasticsuite specific table.

Regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question safe to test PR can be tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants