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

Deeplink metamask://wc?uri=wc:..... broken in versión 6.5.0 #6457

Closed
jcorreapa opened this issue May 24, 2023 · 16 comments
Closed

Deeplink metamask://wc?uri=wc:..... broken in versión 6.5.0 #6457

jcorreapa opened this issue May 24, 2023 · 16 comments
Labels
deeplinks Deeplinks related issue or bug team-sdk SDK team

Comments

@jcorreapa
Copy link

jcorreapa commented May 24, 2023

Describe the bug
I am developing a native mobile application in Flutter, for this I use the library [https://pub.dev/packages/walletconnect_dart], at the moment of making the connection and generating the Uri, a uri of the type 'wc:... @1?bridge=https%3A%2F%2Fg.bridge.walletconnect.org&key=....'.

Before the current version 6.5.0, to open the metamask app, it worked perfectly for IOS and Android with the following structure: launchUrlString('metamask://wc?uri=$uri', mode: LaunchMode.externalApplication). But with the current version it stopped working, because although the app is opened, the connection modal is not shown.

It is not a problem of the library used, because when opening the application with the following structure: launchUrlString(uri, mode: LaunchMode.externalApplication), the modal is displayed. Which is not suitable for me, as this will open any wallet that supports the wc protocol, and my app only supports Metamask mobile.

Screenshots

deeplink_example.mp4

To Reproduce
Steps to reproduce the behavior

Smartphone (please complete the following information):

Device: Samsumng S20
OS: Android
App Version: v6.5.0

@idelburgo-iomob
Copy link

You posted the same code for "incorrect" and "correct", which is the code that is working for you? I have similar problem too, but with a native Android app (Kotlin)

@anaamolnar anaamolnar added deeplinks Deeplinks related issue or bug team-sdk SDK team labels May 25, 2023
@ContrastPro
Copy link

This code dost work correctly and don't show modal in my case only on some Android devices:
launchUrlString('metamask://wc?uri=$uri', mode: LaunchMode.externalApplication)

But this code works perfectly
launchUrlString('$uri', mode: LaunchMode.externalApplication)

@ContrastPro
Copy link

ContrastPro commented May 25, 2023

To open Metamask, and not other crypto wallets, I like many developers, use the special prefix "metamask://wc?uri=$uri". But it seems that after a recent update, the Metamask team changed some data associated with deep links, and they stopped working correctly.

@idelburgo-iomob
Copy link

I have tried all these (Im in a native app in Kotlin):

  • link as "metamask://wc?uri=$uri" -> opens metamask but no modal to connect
  • link as "wc://wc?uri=$uri" -> let me select which wallet to use, selecting metamask but no modal to connect
  • link as "$uri" -> let me select which wallet to use, selecting metamask returns to my app with instant "Session rejected"

This error broke my implementation that was working prior to this version.

Any suggestion to make it work again?

@jcorreapa
Copy link
Author

You posted the same code for "incorrect" and "correct", which is the code that is working for you? I have similar problem too, but with a native Android app (Kotlin)

It's true, I just edited it. The one that really works is the: launchUrlString(uri, mode: LaunchMode.externalApplication)

@ioridev
Copy link

ioridev commented Jun 11, 2023

    launchUrlString(
            _uri,.
            mode: LaunchMode.externalApplication,.
          ); 

I can connect on Android, but no connection modal on iOS. I've been having trouble for a month.
When will it be fixed?

@adrianotadao
Copy link

adrianotadao commented Jun 26, 2023

any update on this?

@adrianotadao
Copy link

@unicorndev99 are you sure about it? It does not work for me. 😢

@ContrastPro
Copy link

ContrastPro commented Jun 28, 2023

If someone managed to connect to Metamask using the Flutter WalletConnect library, please write an example of how you implemented it! Or attach links to instructions for connecting WalletConnect with Metamask.

@solsticesolutions
Copy link

@unicorndev99 the issue I am seeing is universal links are working but deep links are not.
Is it working for you with a deep link on iOS?

@ContrastPro
Copy link

Guys check this issue we discuss some points how to resolve problems with connections.

@christopherferreira9
Copy link
Contributor

Closing this issue as it seems that the new MetaMask wallet solved it along side this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deeplinks Deeplinks related issue or bug team-sdk SDK team
Projects
None yet
Development

No branches or pull requests

10 participants