-
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
[Forwardport] Magento Catalog - fix custom option type text price conversion for mu… #22760
[Forwardport] Magento Catalog - fix custom option type text price conversion for mu… #22760
Conversation
Hi @ihor-sviziev. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @rogyar, thank you for the review. |
646ef8e
to
ccecc7c
Compare
✔️ QA Passed |
…ext price conversion for mu… magento#22760
Hi @ihor-sviziev, thank you for your contribution! |
Original Pull Request
#22016
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