Skip to content

fix: Update network nicknames and block explorer URLs#9005

Open
FrederikBolding wants to merge 3 commits into
mainfrom
fb/update-network-nicknames-and-block-explorer
Open

fix: Update network nicknames and block explorer URLs#9005
FrederikBolding wants to merge 3 commits into
mainfrom
fb/update-network-nicknames-and-block-explorer

Conversation

@FrederikBolding
Copy link
Copy Markdown
Member

@FrederikBolding FrederikBolding commented Jun 4, 2026

Explanation

Updates all network nicknames to match overrides used by the clients. Additionally, ensure block explorer URLs are populated by default so the client doesn't have to do it.

References

https://consensyssoftware.atlassian.net/browse/WPC-1051

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
UI-facing display strings and default network metadata only; no auth, RPC, or transaction logic changes.

Overview
Aligns default built-in network display names with client overrides by updating NetworkNickname in @metamask/controller-utils (e.g. Ethereum MainnetEthereum, BSC MainnetBNB Chain, Optimism MainnetOP).

NetworkController now seeds default Infura network configurations with block explorer URLs from BUILT_IN_NETWORKS (blockExplorerUrls plus defaultBlockExplorerUrlIndex: 0) instead of leaving them empty, so clients do not need to patch explorers separately.

Changelogs and NetworkController test snapshots/error strings were updated to match the new names and explorer fields.

Reviewed by Cursor Bugbot for commit f166437. Bugbot is set up for automated code reviews on this repo. Configure here.

@FrederikBolding FrederikBolding marked this pull request as ready for review June 4, 2026 13:31
@FrederikBolding FrederikBolding requested review from a team as code owners June 4, 2026 13:31
@FrederikBolding FrederikBolding deployed to default-branch June 4, 2026 13:32 — with GitHub Actions Active
Copy link
Copy Markdown
Member

@mikesposito mikesposito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current client overrides do not affect the initial state of NetworkController, so AFAIK these names are effectively used on onboarding users.

They are also the same as the client config API: https://client-config.api.cx.metamask.io/v1/config/networks

Not an issue, just something I thought it would be good to point out as it's unclear who should be the source of truth to align to

@FrederikBolding
Copy link
Copy Markdown
Member Author

Current client overrides do not affect the initial state of NetworkController, so AFAIK these names are effectively used on onboarding users.

They are also the same as the client config API: https://client-config.api.cx.metamask.io/v1/config/networks

Not an issue, just something I thought it would be good to point out as it's unclear who should be the source of truth to align to

The changes here should also only apply to onboarding users, it is simply meant to standardize that across clients and ensure this is the default behaviour.

@FrederikBolding FrederikBolding requested a review from mcmire June 4, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants