Skip to content

fix(network-enablement-controller): add Tempo to POPULAR_NETWORKS so "All popular networks" selects correctly #9069

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.

Sign up for GitHub

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

Open
vinnyhoward wants to merge 4 commits into
base: main
Choose a base branch
from
+2 −0
Open

fix(network-enablement-controller): add Tempo to POPULAR_NETWORKS so "All popular networks" selects correctly #9069

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e583bca
feat: add tempo to network enablement
vinnyhoward Jun 9, 2026
fa5ee53
fix: update changelog
vinnyhoward Jun 9, 2026
98f707f
fix: update pr placeholder with real PR
vinnyhoward Jun 9, 2026
7ced793
Merge branch 'main' of github.com:MetaMask/core into feat-tmcu-857-ad…
vinnyhoward Jun 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/network-enablement-controller/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Added

- Export `NetworkEnablementControllerStateChangeEvent` type from the package root ([#9084](https://github.com/MetaMask/core/pull/9084))
- Added Tempo (`0x1079`) to `POPULAR_NETWORKS`, so that `enableAllPopularNetworks()` enables it alongside the other popular networks and keeps the controller in sync with the client's popular networks list ([#9069](https://github.com/MetaMask/core/pull/9069))

### Changed

Expand Down
1 change: 1 addition & 0 deletions packages/network-enablement-controller/src/constants.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,4 +16,5 @@ export const POPULAR_NETWORKS: readonly Hex[] = [
'0x8f', // Monad (143)
'0x10e6', // MegaETH (4326)
'0x13b2', // Arc (5042)
'0x1079', // Tempo (4217)

@Prithpal-Sooriya Prithpal-Sooriya Jun 10, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just wanna double check. Popular networks doesn't mean default right? It should appear as an "additional network"?

Just being overly cautious :)

@vinnyhoward vinnyhoward Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

In this context it is a popular network, I've made that assumption because it was recently added into mobile and it appears as popular. I've attached a video below showing Tempo in popular networks. Also, its showcasing the bug explained in the PR description @Prithpal-Sooriya

If this is incorrect I'm happy to fix the issue or close the PR and come up with another solution. I just need more context on the Tempo network addition

Simulator.Screen.Recording.-.iPhone.17.-.2026-06-10.at.15.38.15.mov

];
Loading