-
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
Magento Catalog - fix custom option type text price conversion for mu… #22016
Magento Catalog - fix custom option type text price conversion for mu… #22016
Conversation
…lti currency website
Hi @oleksii-lisovyi. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @ihor-sviziev, thank you for the review. |
Hi @oleksii-lisovyi, in order to focus on the 2.3 release line we are accepting PRs to 2.3-develop first. Could you please create PR with the same changes to 2.3-develop and we will process current PR once it will be merged. As for now I'm putting this PR on hold before PR to 2.3-develop branch will be merged. Thank you! |
Hi @sidolov, thank you for the review. |
✔️ QA passed |
Hi @oleksii-lisovyi, thank you for your contribution! |
Description
Previously I already fixed similar issue in the PR #18225, but the fix doesn't cover all the custom option types, e.g. type "Field" still has that issue with price conversion.
It's reproduced on the following Magento setup with multi-currency configuration:
Fixed Issues (if relevant)
No related issues found.
Manual testing scenarios
Steps to reproduce
Expected result
Custom option price is not changed, because no actual changes were done.
Actual result
Custom option price is updated according to USD/CAD currency rate.
Contribution checklist