Skip to content

Conversation

@struong
Copy link

@struong struong commented Jan 6, 2026

Description

Add Tempo testnet network icons and assets to MetaMask extension.

  • Add TEMPO_TESTNET chain ID for network identification
  • Display Tempo network icon in the extension

Open in GitHub Codespaces

Changelog

CHANGELOG entry: Added Tempo testnet network icon and assets

Related issues

Fixes:

Manual testing steps

  1. Build the extension with yarn start
  2. Add Tempo testnet network (Chain ID: 0xa5bd)
  3. Verify the Tempo network icon displays correctly in the network selector

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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.

[CHAIN_IDS.MEGAETH_MAINNET]: MEGAETH_MAINNET_IMAGE_URL,
[CHAIN_IDS.NEAR]: NEAR_IMAGE_URL,
[CHAIN_IDS.NEAR_TESTNET]: NEAR_IMAGE_URL,
[CHAIN_IDS.TEMPO_TESTNET]: TEMPO_TOKEN_IMAGE_URL,
Copy link

Choose a reason for hiding this comment

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

Missing native token image registration for Tempo testnet

Low Severity

The TEMPO_TESTNET chain is added to CHAIN_ID_TO_NETWORK_IMAGE_URL_MAP for network icons, but not to CHAIN_ID_TOKEN_IMAGE_MAP which is used for native token images in components like the asset picker. Other similar testnets (e.g., MONAD_TESTNET, SOPHON_TESTNET, MEGAETH_TESTNET) are registered in both maps. This omission could cause the native token image to be missing/undefined when displaying Tempo testnet assets.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants