walletconnect-dart-sdk icon indicating copy to clipboard operation
walletconnect-dart-sdk copied to clipboard

Does this library support ethereum L2s / switching chains?

Open jagged91 opened this issue 2 years ago • 6 comments

See the title! For example, Polygon

jagged91 avatar Mar 08 '22 17:03 jagged91

I have tested connecting and performing SignedTypeData requests on Polygon against the Rainbow wallet. Users can switch between networks and your dart client will see it in the onSessionUpdate: callbacks etc. I don't see why the other functions would not work - I just didn't need them in my app.

bobwith2bees avatar Mar 09 '22 13:03 bobwith2bees

@bobwith2bees can you try using metamask, i facing issue that the dart side doesn't receive onSessionUpdate when I changed the chain.

ekasetiawans avatar Mar 10 '22 10:03 ekasetiawans

@ekasetiawans Oh Yay.. Metamask has an updated Android build as of this week!

I don't see how to change the network for the specific Wallet Connect session in progress. Only to disconnect. (Settings > Experimental > WalletConnect > View Sessions > long press the session.)

If you select the network before connecting - I see the network correctly. (Rinkeby... I don't know how to enable Polygon networks with the mobile app just yet.)

bobwith2bees avatar Mar 10 '22 16:03 bobwith2bees

I am seeing an issue with Metatmask not returning a signTypedData request. The connection to the wallet is fine. The behavior is the same on both Android and IOS. (There was a recent release 4.1.1 - Android / 4.2.2 IOS in the last week. )

Metamask has a few WalletConnect issues still open (even if the current release notes say there were improvements to WalletConnect. ). https://github.com/MetaMask/metamask-mobile/issues?q=is%3Aissue+is%3Aopen+walletconnect

bobwith2bees avatar Mar 12 '22 13:03 bobwith2bees

Cross reference to the Metamask mobile issue I filed: https://github.com/MetaMask/metamask-mobile/issues/3887

bobwith2bees avatar Mar 13 '22 15:03 bobwith2bees

FYI - Looks like my issue was on the WalletConnect repo side, not Metamask. WalletConnect was dropping the V4 version number, and the Metamask Client didn't accept it. (Other clients had a fallback to v4 evidently.)

Fix:

  • https://github.com/WalletConnect/walletconnect-monorepo/pull/1191 Integrated Code Release:
  • https://github.com/WalletConnect/walletconnect-monorepo/releases/tag/1.8.0

bobwith2bees avatar Oct 24 '22 14:10 bobwith2bees