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

[Bug - SDK] iOS 17 Wallet doesn't send back to dapp #7165

Closed
christopherferreira9 opened this issue Sep 7, 2023 · 6 comments
Closed

[Bug - SDK] iOS 17 Wallet doesn't send back to dapp #7165

christopherferreira9 opened this issue Sep 7, 2023 · 6 comments
Labels
Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing team-sdk SDK team type-bug Something isn't working

Comments

@christopherferreira9
Copy link
Contributor

Describe the bug
On iOS 17 (beta as of today) the MetaMask Wallet doesn't send the user back to the dapp as it should.
This seems to be related to Minimmizr.

Screenshots

265540228-28c25eb9-f2f4-420e-bf58-1583fcbb9067.mov

To Reproduce

  1. Connect to any dapp via deeplink
  2. See error

Expected behavior
The Wallet sends the user back to the dapp after connecting

Smartphone (please complete the following information):


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?
@christopherferreira9 christopherferreira9 added type-bug Something isn't working team-sdk SDK team labels Sep 7, 2023
@anaamolnar anaamolnar added the Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking label Sep 14, 2023
@sethkfman sethkfman added Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing and removed Sev2-normal An issue that may lead to users misunderstanding some limited risks they are taking labels Oct 17, 2023
@xrdavies
Copy link

Are there any updates?

@codepgq
Copy link

codepgq commented Nov 1, 2023

any updates?

@m1heng
Copy link

m1heng commented Nov 4, 2023

any updates? need more attention into this.

@harry12342
Copy link

any updates?

@christopherferreira9
Copy link
Contributor Author

Hi all! Unfortunately we are not able to resolve this due to OS limitations on the iOS side.
We have implemented a new modal to ask the user to go back to the dapp manually as a workaround. This as been merged and will be available in the next Mobile Wallet version.

@alienzhangyw
Copy link

Hi all! Unfortunately we are not able to resolve this due to OS limitations on the iOS side. We have implemented a new modal to ask the user to go back to the dapp manually as a workaround. This as been merged and will be available in the next Mobile Wallet version.

Similar problem here on Android since V7.3.0 and is still no been fixed yet. (#6943)

  1. Transaction request on dapp (using wagmi’s useContractWrite) do not call up Metamask, you have to manually switch to Metamask mobile.
  2. Metamask do not redirect back to chrome or other Android browsers after signing transactions. Sometimes it only minimize, sometimes nothing happens. You have to manually switch back too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sev1-high An issue that may have caused fund loss or access to wallet in the past & may still be ongoing team-sdk SDK team type-bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants