-
-
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
feat(ramp): add deeplink handler #9153
Conversation
|
|
LGTM for QA ✅ I verified the new and existing deeplinks |
|
|
We are investigating this Bitrise error (it's caused by #9345).
This has been retried, and we have confirmed that the error is caused by a bad cache entry |
|
|
Description
This PR:
/buy
and/sell
deeplinks actions (in addition to previous/buy-crypto
and/sell-crypto
). This meansmetamask://buy
,metamask://sell
,https://metamask.app.link/buy
andhttps://metamask.app.link/sell
will be handled by the deeplink manager.handleRampUrl
handler: This will help ramp team to handle more complex URLs. For example/buy?chainId=1&token=0x4946fcea7c692606e8908002e55a582af44ac121
)Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/RAMPS-1510
Manual testing steps
New links should open the buy or sell feature:
Existing functionality must not have changed.
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist