Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot connect from MetaMask mobile via WalletConnect V2 #2435

Closed
ElvisKrop opened this issue Aug 23, 2023 · 4 comments · Fixed by #2444
Closed

Cannot connect from MetaMask mobile via WalletConnect V2 #2435

ElvisKrop opened this issue Aug 23, 2023 · 4 comments · Fixed by #2444
Assignees

Comments

@ElvisKrop
Copy link
Contributor

Bug description

I'm unable to establish a connection via WalletConnect V2 from MetaMask mobile on https://app.safe.global.
Just to proof that MetaMask on my mobile is working well, I was able to complete the same steps on https://app.uniswap.org.

Environment

  • Browser: Chrome / Brave / Firefox (I think any)
  • Wallet: WalletConnect V2 + Mobile MetaMask on Android
  • Chain: Any

Steps to reproduce

  1. Go to https://app.safe.global;
  2. Click on "Connect Wallet";
  3. Select "WalletConnect" option from the modal;
  4. open the MetaMask on your mobile phone;
  5. scan the proposed qr-code from the desktop screen;

Expected result

User should be able to connect the MetaMask mobile wallet via WalletConnect.

Obtained result

Unable to establish a connection via WalletConnect V2 from MetaMask mobile.

Screenshots

image

There are a few errors in the console:
image

@ElvisKrop ElvisKrop added the bug label Aug 23, 2023
@github-project-automation github-project-automation bot moved this to New issues in Safe{Wallet} Aug 23, 2023
@katspaugh katspaugh moved this from New issues to Ready for QA in Safe{Wallet} Aug 23, 2023
@katspaugh
Copy link
Member

@francovenica could you check if you can reproduce this?

@francovenica francovenica moved this from Ready for QA to QA in progress in Safe{Wallet} Aug 23, 2023
@francovenica
Copy link
Contributor

Yes, is like that in Prod.

@katspaugh katspaugh moved this from QA in progress to New issues in Safe{Wallet} Aug 24, 2023
@katspaugh
Copy link
Member

I've also reproduced this with the iOS MetaMask.

@katspaugh
Copy link
Member

Looks like it's a web3-onboard bug, looking for a solution.

@katspaugh katspaugh moved this from In Progress to Peer Review in Safe{Wallet} Aug 25, 2023
@katspaugh katspaugh moved this from Peer Review to Ready for QA in Safe{Wallet} Aug 25, 2023
@francovenica francovenica moved this from Ready for QA to QA in progress in Safe{Wallet} Aug 25, 2023
@francovenica francovenica moved this from QA in progress to QA done in Safe{Wallet} Aug 25, 2023
@github-project-automation github-project-automation bot moved this from QA done to Done in Safe{Wallet} Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants