tokamak-network / tokamak-bridge

An interface designed to bridge assets between Titan and Ethereum chains.
https://app.bridge.tokamak.network
7 stars 0 forks source link

Support for WalletConnect Integration #104

Open NegruGeorge opened 3 months ago

NegruGeorge commented 3 months ago

Description: The current implementation of the Tokamak Bridge only supports MetaMask for wallet connections. While MetaMask is a popular choice, adding support for WalletConnect would greatly enhance the user experience by allowing users to connect using a variety of other wallets.

Proposal: I propose integrating WalletConnect into the Tokamak Bridge. WalletConnect is a widely-used protocol that allows users to connect any wallet that supports WalletConnect, such as Trust Wallet, Argent, and many others. This would provide users with greater flexibility in how they interact with the Tokamak Bridge.

Benefits:

Broader Wallet Support: Users can connect using a wide range of wallets beyond MetaMask, which could increase the platform's appeal to a broader audience. Improved User Experience: Offering more wallet options can reduce friction for users who prefer or exclusively use wallets other than MetaMask. Potential for Increased Adoption: Supporting additional wallets could attract more users who are looking for a platform that accommodates their preferred wallet.

Potential Implementation:

Add WalletConnect as a dependency in the project. Update the wallet connection logic to include support for WalletConnect alongside MetaMask. Modify the user interface to allow users to select WalletConnect when connecting to the Tokamak Bridge. Additional Context: While this proposal focuses on integrating WalletConnect, other kits such as RainbowKit or wagmi connect wallet could also be considered for future enhancements. These kits provide advanced features and more flexibility for developers looking to support a variety of wallets with minimal integration effort.

suahnkim commented 1 month ago

Hello @NegruGeorge we think this is a good idea, let us know if you plan to implement this feature (share a deadline).

We do have to update our modals to replace "MetaMask" to "Wallet" though. Let's work on it at some point