-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[5.2][IMPROVEMENT] Adds support for 'dapp/' urls support on 'metamask://' and fixes DL opening to Apple Store #4167
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
May I ask when do you plan to release this fix? This is blocking issue for our team :( |
Hi @dmitrymikheevappus, most likely with version 5.2.0 so probably in a couple of weeks. In the meantime did you see what I wrote here? |
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
0b38660
to
2dfc94c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to deep link using universal deep links and url scheme deep links on both iOS and Android. This guy is 🌮 🌮 🌮
Description
Adds support for 'dapp/' urls support on 'metamask://' scheme.
So that it'll be possible to use links like:
metamask://dapp/google.com
and have the same results ofhttps://metamask.app.link/dapp/google.com
.This is not a deep link so it won't check if the app is installed and if not, take the user to stores to download it.
This is only works if the app is already installed (otherwise the device doesn't know how to deal with
metamask://
links.Reason to use this over Branch.io links is that it does not depend on it and so can overcome some of the limitations of them, like sometimes not opening when programmatically called from dapps (without a user click/tap).
It also fixes the issue related to deep links going to Apple store on iOS even if the app is installed.
Checklist
Issue
Progresses #3965