Skip to content

fix: Fixing test-tokens page#3821

Open
Genesis3800 wants to merge 1 commit into
mainfrom
ccip-test-tokens-faucet-fix
Open

fix: Fixing test-tokens page#3821
Genesis3800 wants to merge 1 commit into
mainfrom
ccip-test-tokens-faucet-fix

Conversation

@Genesis3800
Copy link
Copy Markdown
Collaborator

This pull request significantly refactors wallet connection and network selection logic to support multiple injected EVM wallets (such as MetaMask and Rabby), rather than being hardcoded for MetaMask.

Wallet selection and connection improvements:

  • Generalized wallet detection and selection: The code now supports multiple injected EVM wallets, allowing users to pick their preferred wallet (e.g., MetaMask, Rabby) via a UI selector.

Network dropdown and chain management:

  • The NetworkDropdown component now receives the selected wallet provider as a prop, removing the dependency on a MetaMask-specific hook. This enables network switching and token minting on any compatible EVM wallet.

Error handling and user feedback:

  • Enhanced error handling for wallet connection and network switching.

@Genesis3800 Genesis3800 self-assigned this May 27, 2026
@Genesis3800 Genesis3800 requested review from a team as code owners May 27, 2026 21:49
@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
documentation Ready Ready Preview May 27, 2026 9:50pm

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

👋 Genesis3800, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

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.

1 participant