WalletConnect / WalletConnectFlutterV2

WalletConnect v2 client made in Dart for Flutter.
https://pub.dev/packages/walletconnect_flutter_v2
Apache License 2.0
117 stars 61 forks source link

after connected uniswap, chains list shows unsupported by your wallet #291

Closed leeway1208 closed 6 months ago

leeway1208 commented 6 months ago

I have connected to uniswap. but I can not change the chain. It shows unsupported by your wallet image

quetool commented 6 months ago

Hello @leeway1208! Thanks for the report! This seems to be an issue on Uniswap, allow me some time to investigate.

leeway1208 commented 6 months ago

Hello @leeway1208! Thanks for the report! This seems to be an issue on Uniswap, allow me some time to investigate.

Thx~~~

quetool commented 6 months ago

Hello @leeway1208! Could you try if the same is happening on Pancakeswap?

leeway1208 commented 6 months ago

Hello @leeway1208! Could you try if the same is happening on Pancakeswap?

Yes. Always shows requesting. image

quetool commented 6 months ago

But the different chains are enabled, right?

leeway1208 commented 6 months ago

But the different chains are enabled, right?

Yes

quetool commented 6 months ago

Sounds good. So, the reason why PancakeSwap is showing "Requesting..." could be because you are trying to switch to an unsupported chain (switching from Ethereum to BNB and viceversa shouldn't be a problem). Are you testing trying this out with our Sample Wallet?

Regarding Uniswap issue I'm still looking into it.

quetool commented 6 months ago

Hello @leeway1208! We have found the issue and we notified Uniswap team! But we are also implementing a fix on our side just to overcome this issue in the future and avoid confusions. Thank you so mucho for your report! 🎉 I'll keep you posted once our fix is deployed!

leeway1208 commented 6 months ago

Thanks~

quetool commented 6 months ago

Hello @leeway1208! Could you check again on Uniswap? Thanks!

leeway1208 commented 6 months ago

Hello @leeway1208! Could you check again on Uniswap? Thanks!

I can change the chain now. But list displays the text that approve in wallet. What should I do in next. Thanks image

quetool commented 6 months ago

Yeah so, the fix from Uniswap consist in enabling every chain they support but that doesn't mean you support it. Which is most probably the case here. So if you want to support Base (and every other chain listed there) you should add support on your side. Check how it's done in our sample wallet.

In the coming days another patch for the wagmi connector is going to be release that will enable oney the chains the wallet support and not every chain.

I'll close for now as this issue is fixed, but feel free to open a new issue if you encounter.