web3modal icon indicating copy to clipboard operation
web3modal copied to clipboard

[bug] Integrate TRON chain support for Web3Modal

Open BeycanDeveloper opened this issue 1 year ago • 13 comments

Link to minimal reproducible example

It's not necessary

Summary

Hello, I use WalletConnect in my project for both Tron and EVM-based networks. In doing this, I use the following package provided by Tron.

package

However, since they both try to define the same "web components", I get the following error.

DOMException: Failed to execute 'define' on 'CustomElementRegistry': the name "w3m-button" has already been used with this registry

Therefore, I think you can solve this problem by adding a small condition to the places where these components are defined.

List of related npm package versions

"@wagmi/core": "^1.4.7", "@web3modal/wagmi": "^3.3.2", "viem": "^1.19.11"

Usingi n Tron WalletConnect adapter

"@tronweb3/walletconnect-tron": "2.0.0", "@wagmi/core": "^1.1.0", "@walletconnect/sign-client": "^2.1.4", "@walletconnect/types": "^2.1.4", "@web3modal/ethereum": "^2.4.2", "viem": "^0.3.50",

BeycanDeveloper avatar Dec 02 '23 02:12 BeycanDeveloper

This is not recommended as you might end up with localStorage issues. We're planning to add support for more chains in the future though.

glitch-txs avatar Dec 02 '23 03:12 glitch-txs

I understand that I have moved the codes in the tron package to a separate area right now and I will try to replace them with WalletConnect 1.

BeycanDeveloper avatar Dec 02 '23 04:12 BeycanDeveloper

@glitch-txs Hi~currently, the package @web3modal/ethers and @web3modal/wagmi can both only be used for eip155 namespace. Is there any plan to support other namespace such as tron:0x94a9059e namespace ? Thank you!

way2ex avatar Jan 22 '24 06:01 way2ex

Hi we're working a Solana integration and are planning to add Tron in the future as well, there's no current support for other namespaces atm

glitch-txs avatar Jan 22 '24 10:01 glitch-txs

@glitch-txs how about Bitcoin and maybe for Xverse wallet? WalletConnect really needed for Bitcoin. Also, i want create a open source project MultipleChain i will write with TS can we talk with you about of structure?

BeycanDeveloper avatar Jan 22 '24 11:01 BeycanDeveloper

could you open a separated issue for each of those chains?

glitch-txs avatar Jan 22 '24 11:01 glitch-txs

@glitch-txs yes i can open issue for Bitcoin & Xverse wallet support. But i want talk with you about of a project struct. That's why i contacted with you on Twitter can we talk there?

BeycanDeveloper avatar Jan 22 '24 12:01 BeycanDeveloper

happy to take PRs @BeycanDeveloper but I think we are a bit early to support multi chain in Web3Modal

glitch-txs avatar Jan 22 '24 12:01 glitch-txs

@glitch-txs No it’s not for w3m this is separate open source project it’s my project. I just want talk with you about of this.

BeycanDeveloper avatar Jan 22 '24 12:01 BeycanDeveloper

Hi we're working a Solana integration and are planning to add Tron in the future as well, there's no current support for other namespaces atm

Thanks a lot for your great work!

way2ex avatar Jan 23 '24 01:01 way2ex

i am facing the same issue in my project i have added web3 modal for USDT transfers for multiple mainnet evm chains but for tron i am using "@tronweb3/walletconnect-tron": "^2.0.0", library for trust wallet and i am getting the same issue how can i differentiate the web component

Muzided avatar Mar 13 '24 05:03 Muzided

Hi we're working a Solana integration and are planning to add Tron in the future as well, there's no current support for other namespaces atm

@glitch-txs TRON now has a high market share in the cryptocurrency world, at least higher than Solana. It is recommended to increase support for the TRON network.

hhun avatar May 06 '24 03:05 hhun

@hhun you can use legacy modal version with this package: https://github.com/MultipleChain/tron-walletconnect

BeycanDeveloper avatar May 06 '24 03:05 BeycanDeveloper