-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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: display snap account name #22342
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
ui/components/multichain/account-list-menu/account-list-menu.js
Outdated
Show resolved
Hide resolved
ui/components/multichain/account-list-menu/account-list-menu.js
Outdated
Show resolved
Hide resolved
ff91dc6
to
9fdad04
Compare
07902f9
to
2e38cf9
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #22342 +/- ##
===========================================
+ Coverage 67.90% 67.92% +0.02%
===========================================
Files 1071 1071
Lines 41378 41399 +21
Branches 11112 11123 +11
===========================================
+ Hits 28095 28118 +23
+ Misses 13283 13281 -2 ☔ View full report in Codecov by Sentry. |
ui/components/multichain/account-list-menu/account-list-menu.js
Outdated
Show resolved
Hide resolved
…dated tests and stories
Updated how keyring is derived in wallet-overview Co-authored-by: Gustavo Antunes <17601467+gantunesr@users.noreply.github.com>
Co-authored-by: Gustavo Antunes <17601467+gantunesr@users.noreply.github.com>
60f0140
to
fe9fa3d
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.
Builds ready [fe9fa3d]
Page Load Metrics (1551 ± 141 ms)
Bundle size diffs
|
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.
LGTM. The feature works as expected. Ship it
Description
This feature adds a label for snap accounts with the derived snap name and MetaMask snap icon to the tag on the accounts list and wallet overview UI.
Reason: Account snaps should present a snap tag, including the snap badge and the name of the snap it's derived from.
Improvement: The account-list-menu has been updated to pass a keyring type and optional tag label to child account-list-item components. The wallet-overview screen now shows a tag with the snap name for snap accounts.
Related issues
Fixes: #20819
Manual testing steps
Jest
jest ui/components/multichain/account-list-menu/account-list-menu.test.js
jest ui/components/multichain/account-list-item/account-list-item.test.js
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist