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

Cover image only in the first image #6

Open
davidglezz opened this issue Jan 18, 2021 · 3 comments
Open

Cover image only in the first image #6

davidglezz opened this issue Jan 18, 2021 · 3 comments

Comments

@davidglezz
Copy link
Contributor

davidglezz commented Jan 18, 2021

Current behaviour: all generated images are cover:

<image id_product="4" id="13" cover="1"/>
<image id_product="4" id="14" cover="1"/>
<image id_product="4" id="15" cover="1"/>
<image id_product="4" id="16" cover="1"/>

then, only one image are added to the product:
image

If we modify cover value to be 0

<image id_product="3" id="9" cover="0"/>
<image id_product="3" id="10" cover="0"/>
<image id_product="3" id="11" cover="0"/>
<image id_product="3" id="12" cover="0"/>

image

But then... No cover image
image

EXPECTED OUTPUT: only one image with cover="1"

<image id_product="3" id="9" cover="1"/>
<image id_product="3" id="10" cover="0"/>
<image id_product="3" id="11" cover="0"/>
<image id_product="3" id="12" cover="0"/>
@davidglezz
Copy link
Contributor Author

davidglezz commented Feb 15, 2023

I think this should be labeled as bug.

@davidglezz
Copy link
Contributor Author

PR ready =)

@matks
Copy link
Contributor

matks commented Mar 14, 2023

Sorry @davidglezz usually issues are opened on https://github.com/prestashop/prestashop I dont know why this repository allows opening issues

I ask QA team to see if they can reproduce your problem

@davidglezz davidglezz changed the title Cover image only first image Cover image only in the first image Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants