-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Checkout] Ensured exchange rate gets updated throughout checkout #5900
[Checkout] Ensured exchange rate gets updated throughout checkout #5900
Conversation
NoResponseMate
commented
Aug 26, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | yes |
Related tickets | - |
License | MIT |
4ba747b
to
1ed2faa
Compare
fa1317e
to
131e3ac
Compare
sylius_update_exchange_rate: | ||
on: ["select_payment"] | ||
do: ["@sylius.updater.order.exchange_rate_and_currency", "update"] | ||
args: ["object"] |
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.
Do we need this?
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.
Until #5847 gets merged unfortunately it is necessary, otherwise after the select_payment
transition the cart currency code and exchange rate reset to the default values.
131e3ac
to
8bc158d
Compare
$this->selectPaymentPage, | ||
$this->selectShippingPage, | ||
$this->completePage, | ||
$this->thankYouPage, |
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.
This should not be the case, hm?
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.
Welp, it's a possibility nonetheless 😄
8bc158d
to
fc5aab1
Compare
fc5aab1
to
31ccadd
Compare
Thanks Janek! |
…ate-update [Checkout] Ensured exchange rate gets updated throughout checkout
…ate-update [Checkout] Ensured exchange rate gets updated throughout checkout