-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Fixed issue: Add to Bag within "Click for price" popup not working. #30357
base: 2.4-develop
Are you sure you want to change the base?
Fixed issue: Add to Bag within "Click for price" popup not working. #30357
Conversation
Hi @sanganinamrata. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento run Static Tests |
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 @sanganinamrata. Thank you for your contribution. Could I ask you to cover your changes with an MFTF test, please?
The test will go through the "Steps to reproduce" described in the original issue and assert that we meet the expected behavior.
Thank you!
@rogyar Sorry for the late reply, Actually I haven't written the MFTF test till now. But I've identified this issue on Magento 2.3 and 2.4 both and tested it with the above solution. |
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 @sanganinamrata, thanks for your contribution.
I'm moving this PR to change requested column since it requires test coverage.
@engcom-Foxtrot @engcom-Hotel @engcom-Charlie can you help with test coverage on this?
@magento run all tests |
@sanganinamrata Please resolve with the conflicts to move ahead with this PR |
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.
Please resolve the conflicts on the PR
Description (*)
Fixed Add to cart functionality within *Click for price* popup on any product listings.Fixed Issues (if relevant)
Contribution checklist (*)