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 settings to get main parent image #138

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

JoseManuelFV
Copy link

When main variation image is empty, not sending main image. With this configuration you can control this behavior

To reproduce the “error”, you can use the shopping_feed_variation_images hook.
When you are using this hook and you have not set a main variation image, it causes the main image to be the 1st image in the gallery and not the main image.
If you have size variations and color variations, it causes the color images to have the correct gallery and the size images to have the wrong gallery (because it is the same gallery as the parent).

As there is no hook to control this behavior or the data of the products, I have chosen to add this pull request.
Logically you can fix this error by adding by hand all the main images to the variations, but this becomes unrealistic when you have many products with many variations.

When main variation image is empty, not sending main image. With this configuration you can control this behavior
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.

1 participant