-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
refactor: remove global network usage from petnames #12257
refactor: remove global network usage from petnames #12257
Conversation
Bitrise✅✅✅ Commit hash: b4222b3 Note
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@metamask/[email protected] |
Quality Gate passedIssues Measures |
Bitrise✅✅✅ Commit hash: b3d8184 Note
|
Description
Remove all usage of the global network from the
Name
component and related display name hooks.Specifically:
selectChainId
selector.variation
property inName
component anduseDisplayName
hook.imageUri
property toIdenticon
component.chainId
property toAddress
component.useTokenListEntry
hook touseERC20Tokens
.selectERC20TokensByChain
selector.useTokenList
hook.Related issues
Fixes: #1987
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist