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

Rejecting Connection Request from WalletConnect Mobile App Doesn't Redirect Back #6926

Closed
Luzzotica opened this issue Jul 28, 2023 · 7 comments
Labels
Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking team-sdk SDK team type-bug Something isn't working WalletConnect WalletConnect related issue or bug

Comments

@Luzzotica
Copy link

Describe the bug
If I deep link to MetaMask using WalletConnect and accept the connection, then MetaMask redirects back to my dapp once it's approved the session.
However, if I cancel the session it doesn't deep link back.

Screenshots
None

To Reproduce

  1. Connect using this website on your mobile device
  2. Tap on MetaMask
  3. Reject/Cancel the connection request
  4. MetaMask doesn't redirect

Expected behavior
MetaMask redirects even when the session is canceled/rejected.

Smartphone (please complete the following information):

  • Device: iPhone 14 Pro (I think)
  • OS: iOS 16
  • App Version 7.3.0 (1144)

to be added after bug submission by internal support / PM
Severity

  • How critical is the impact of this bug on a user?
  • Add stats if available on % of customers impacted
  • Is this visible to all users?
  • Is this tech debt?
@Luzzotica Luzzotica added awaiting-metamask PR or issue awaiting feedback from metamask team needs-triage Issues that require triage type-bug Something isn't working labels Jul 28, 2023
@Luzzotica Luzzotica changed the title Canceling Connection Request from Mobile App Doesn't Redirect Back Rejecting Connection Request from WalletConnect Mobile App Doesn't Redirect Back Jul 28, 2023
@anaamolnar anaamolnar added team-sdk SDK team Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking WalletConnect WalletConnect related issue or bug and removed needs-triage Issues that require triage awaiting-metamask PR or issue awaiting feedback from metamask team labels Jul 30, 2023
@anaamolnar
Copy link

Hello, @Luzzotica. Thanks for reporting! I will pass this on to the team.

@ldkfreetoplay
Copy link

I have the same problem, do you have a solution? @anaamolnar

@mertcankose
Copy link

same

@abretonc7s
Copy link
Contributor

Hi @Luzzotica ,
We are investigating the issue and will get back to you shortly. Thanks for reporting.

@siosio34
Copy link

siosio34 commented Oct 6, 2023

I have the same problem

@christopherferreira9
Copy link
Contributor

Hi all! We are working on a fix for this issue and it'll be available in the coming releases.
Also, quick reminder that due to iOS limitations we're not able to redirect the user back to the dapp from iOS17 forward.

@christopherferreira9
Copy link
Contributor

christopherferreira9 commented Jan 22, 2024

Closing this for now until there's no option for us to deal with it on iOS 17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking team-sdk SDK team type-bug Something isn't working WalletConnect WalletConnect related issue or bug
Projects
None yet
Development

No branches or pull requests

7 participants