-
Notifications
You must be signed in to change notification settings - Fork 602
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
feat(checkout): MCU-427 Display modal before switching currencies #1668
Conversation
Autotagging @bigcommerce/storefront-team @davidchin |
8b2c5f7
to
49c4c36
Compare
@mjhmatt please rebase |
6a3eb56
to
5377e15
Compare
4dfc3a0
to
6cb9cb6
Compare
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.
All good, nice work! 👍
assets/js/theme/global.js
Outdated
@@ -22,7 +22,7 @@ export default class Global extends PageManager { | |||
} = this.context; | |||
cartPreview(secureBaseUrl, cartId); | |||
quickSearch(); | |||
currencySelector(); | |||
currencySelector(this.context.cartId); |
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.
🍹 as cartId is already destructured above, you could just reference cartId
dc3c467
to
d0b70f0
Compare
@mjhmatt @davidchin is this good to go ? |
e178995
to
f582f77
Compare
Sorry for the delay. Yes @junedkazi this is finished now, was just waiting to finalise the backend PR. |
What?
Before switching currencies in the case of a cart containing a gift certificate/promotion we will need to notify the customer as they may be removed from cart once the currency is changed.
Tickets / Documentation
https://jira.bigcommerce.com/browse/MCU-427
Screenshots (if appropriate)
@bigcommerce/checkout