-
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
[Backport] Fix configurable dropdown showing tax incorrectly in 2.3-develop #22535
[Backport] Fix configurable dropdown showing tax incorrectly in 2.3-develop #22535
Conversation
… decreases on the configurable attribute dropdown. This fixes a bug introduced in PR magento#17695 as detailed on issue magento#22270
Hi @amol2jcommerce. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@magento-engcom-team 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. |
Hi @dmytro-ch, thank you for the review. |
✔️ QA passed |
Hi @amol2jcommerce, thank you for your contribution! |
…ctly in 2.3-develop #22535
Original Pull Request
#22466
Description (*)
This pull request addresses an issue described in #22270 and introduced in #17695.
The issue is that the price increase calculation was previously based upon the configurable basePrice vs the optionFinalPrice rather than the configurable finalPrice vs the optionFinalPrice. This meant that the tax amount was also been added to the price difference calculation for each of the configurable options incorrectly.
Fixed Issues (if relevant)
Manual testing scenarios (*)all Magento 2.
Contribution checklist (*)