walletconnect-monorepo
walletconnect-monorepo copied to clipboard
WalletConnect
Two issues: walletconnect and metamask mobile
Describe the bug
Hello, I am using WalletConnect and Metamask Mobile and have the following two issues.
First, Switchchain does not work properly. When you send a Switchchain request and see the chain request window on MetaMask Mobile, if you reject it, the request will not be completed, and even if you accept it, the chain change within the mobile does not work properly, so you have to change the chain within MetaMask Mobile and then process the request.
Second, when sending a sign request while connected to a chain other than Ethereum (ex polygon), there was an issue where the chain was unnecessarily requested to be changed to Ethereum.
These two issues were so critical that we had no choice but to give up on introducing WalletConnect. Is there any progress on these two issues?
Links to related issues are below. https://github.com/MetaMask/metamask-mobile/issues/8380 https://github.com/MetaMask/metamask-mobile/issues/7206 https://github.com/MetaMask/metamask-mobile/issues/7023
SDK Version (if relevant)
- Client: [e.g. JS, Swift, Kotlin]
- Version [e.g. 22]
To Reproduce Steps to reproduce the behavior:
- Go to '...'
- Click on '....'
- Scroll down to '....'
- See error
Expected behavior A clear and concise description of what you expected to happen.
Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
Smartphone (please complete the following information):
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]
Additional context Add any other context about the problem here.
Hi @siosio34, thanks for the report, we'll try to contact MetaMask and suggest these fixes
@glitch-txs
Other than that, walletconnect provides a better user experience. Thank you to the walletconnect team for creating and maintaining this product.
Hi @siosio34 just wanted to give a heads up we've been working together with MetaMask to address the most relevant issues and a new release with a UX improvement around chain switching and else are coming eventually.
Hey @siosio34 the new MetaMask version should be released early next month with these fixed 👍
I think what you said is included in https://github.com/MetaMask/metamask-mobile/releases/tag/v7.21.0. Is this correct?
@glitch-txs
Mmm I think not, @ganchoradkov do you remember which one is the PR for the MetaMask update? But should be on the next release I think