-
-
Notifications
You must be signed in to change notification settings - Fork 256
feat: update network nicknames #7045
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
base: main
Are you sure you want to change the base?
Conversation
|
@metamaskbot publish-preview |
|
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions. |
|
@metamaskbot publish-preview |
|
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions. |
638710b to
037e364
Compare
|
@metamaskbot publish-preview |
|
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions. |
037e364 to
84f9060
Compare
|
@metamaskbot publish-preview |
|
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions. |
Explanation
Popular Network nicknames are incorrect and they are currently being overwritten in both clients during onboarding.
Here we can see how both clients override the names:
https://github.com/MetaMask/metamask-mobile/blob/main/app/core/Engine/controllers/network-controller-init.ts#L55
https://github.com/MetaMask/metamask-extension/blob/main/app/scripts/controller-init/network-controller-init.ts#L83
This PR will make that code redundant and better reflect the initial state.
Here are the draft PRs for both clients. Once this is merged and released the changes will be merged.
MetaMask/metamask-mobile#22137
MetaMask/metamask-extension#37518
References
Related to MetaMask/metamask-extension#37478
Checklist
Note
Updates built-in network nicknames (e.g., Ethereum, Base, Arbitrum, BNB Chain, OP, Polygon, Sei) and aligns tests/error messages; adds changelog entry.
NetworkNicknamemappings insrc/types.ts:Mainnet→Ethereum,Base Mainnet→Base,Arbitrum One→Arbitrum,BSC Mainnet→BNB Chain,Optimism Mainnet→OP,Polygon Mainnet→Polygon,Sei Mainnet→Sei.NetworkController.test.tsto reflect new nicknames.Written by Cursor Bugbot for commit 84f9060. This will update automatically on new commits. Configure here.