-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Resolved configurable product image not showing as per selected options #24383 #24384
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
Conversation
Hi @rani-webkul. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
@magento give me test instance |
Hi @dmytro-ch. Thank you for your request. I'm working on Magento instance for you |
Hi @dmytro-ch, here is your new Magento instance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rani-webkul, unfortunately, the issue is still reproducible in your working branch according to the steps to reproduce mentioned in PR description.
@dmytro-ch as I checked, the actual issue is in magento 2.3-develop branch. this commit is working fine on Magento 2.3.2 version and the issue is, in magento 2.3-develop configurable product selected option image is not showing in any condition. |
Hi @rani-webkul, Could you configure product on test instance? I just tried on https://pr-24384.instances.magento-community.engineering/test.html - it doesn't work : ( |
@rani-webkul sorry for confusing. I mist misunderstood your comment at first. Looks like issue when none of images are selected for configurable products - blocks this PR. Could you report it? |
Hi @ihor-sviziev, |
Here: https://github.com/magento/magento2/issues/new/choose :) |
Hi @ihor-sviziev , |
Thank you for report! We'll keep this Pull Request "on hold" till #24571 will be fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ihor-sviziev Actual issue is #24383 |
It worked for me even without changes from this PR just because of changes from #23033 (were merged 3 months ago). I just cheked it on following test instance that was created from 2.3-develop: Here is video: test config - Google Chrome 2019-09-13 09-37-39.zip |
Hi @rani-webkul, I'm closing this PR as this issue was already fixed in #23033. Feel free to add any comments if you have any. |
Hi @rani-webkul, thank you for your contribution! |
Resolved configurable product image not showing as per selected options #24383
Description (*)
Fixed Issues (if relevant)
Manual testing scenarios (*)
5 . Then select any option it's not showing the actual image, it's showing previous image.
Questions or comments