fix(walletconnect): label chain picker entries Mainnet vs Testnet#89
Merged
Conversation
The WalletConnect / AppKit chain selector showed bare names "Arbitrum One"
and "Arbitrum Sepolia" because @reown/appkit/networks (viem) provides only
the canonical chain name. New users with no crypto background can't tell
at a glance which chain is real money — same picker, two entries that
look interchangeable, real ANT on one side and play tokens on the other.
Spread + override the `name` property before passing networks to
WagmiAdapter and createAppKit. The wagmi/viem internals key off chain id,
not name, so renaming is purely cosmetic for the picker UI.
The Settings direct-wallet <select> dropdown already has the labels
hardcoded ("Arbitrum Sepolia (testnet)" / "Arbitrum One (mainnet)"), so
it's a separate surface that doesn't need changes.
Closes the GitHub issue #39 and related Linear ticket. The desktop
sidebar's amber DEVNET / SEPOLIA TESTNET badge is a different surface
again — that's a status indicator showing which chain is currently active,
not a picker.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The WalletConnect / AppKit chain selector showed bare names "Arbitrum One" and "Arbitrum Sepolia" because `@reown/appkit/networks` (viem) provides only the canonical chain name. New users with no crypto background can't tell at a glance which chain is real money — same picker, two entries that look interchangeable, real ANT on one side and play tokens on the other.
Spread + override the `name` property before passing networks to `WagmiAdapter` and `createAppKit`. wagmi/viem internals key off chain id, not name, so renaming is cosmetic for the picker UI only.
Closes #39.
Notes on adjacent surfaces