-
Notifications
You must be signed in to change notification settings - Fork 69
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
Reorganise payments overview currency switcher, welcome message and balances card #8791
Reorganise payments overview currency switcher, welcome message and balances card #8791
Conversation
Test the buildOption 1. Jetpack Beta
Option 2. Jurassic Ninja - available for logged-in A12s🚀 Launch a JN site with this branch 🚀 ℹ️ Install this Tampermonkey script to get more options. Build info:
Note: the build is updated when a new commit is pushed to this PR. |
Size Change: +1.43 kB (0%) Total Size: 1.25 MB
ℹ️ View Unchanged
|
…pdate/8765-payments-overview-currency-select
…pdate/8765-payments-overview-currency-select
…ence from `CustomSelectControl`
…pdate/8765-payments-overview-currency-select
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.
I have tested these changes as well as implemented & tested #8915 on this branch, its working seamlessly.
I'll fix a PW e2e test had I've inadvertently broken with these changes
✅ Fixed in c6da1bf |
Fixes #8790
Important
This pull request is based on PR #8896 to use the new select component introduced in that PR. Please merge it to
develop
before merging this PR.Caution
These changes are not behind a feature flag.
Changes proposed in this Pull Request
This PR replaces the Payments Overview currency-switching tabs with a new currency selector dropdown.
View design.
Note
A subsequent issue #8490 will handle the Payments Activity Card currency-switching, which is not part of this PR
currency.select.component.demo.mov
Before/After
Before
After
After – stores with a single deposit currency
TODO
_change
to clarify merchant intentTesting instructions
Tip
This PR is currently available to test on the Helix WooExpress demo site. Ask a team Helix member for access.
wcpay_overview_currency_select_change
is recordedselected_currency
query param reflects the currently selected currencyselected_currency
query param from the URL and navigate to it. Ensure the default deposit currency is then selected.selected_currency
query param to the URL (e.g.selected_currency=123
) and navigate to it. Ensure the default deposit currency is then selected.npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge
wcpay_overview_currency_select_change
https://github.com/Automattic/tracks-events-registration/pull/2467wcpay_overview_balances_currency_tab_click
https://github.com/Automattic/tracks-events-registration/pull/2467