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

Ability to buy tokens with Wyre #15992

Merged
merged 3 commits into from
Sep 28, 2022
Merged

Conversation

nikoferro
Copy link
Member

@nikoferro nikoferro commented Sep 27, 2022

Explanation

This extends this already merged PRs: #15551 #15924
and adds Wyre as an extra onramp provider for tokens

Current State: Only native currencies can be bought through Wyre using the buy button
After Change: Ability to purchase tokens defined in BUYABLE_CHAINS_MAP using Wyre

More Information

This depends on a swap-api changed which has already been deployed, this is why it hasn't been included on previous PRs

Screenshots/Screencaps

Before

Screenshot 2022-09-27 at 12 03 01

After

Screenshot 2022-09-27 at 12 02 31

Manual Testing Steps

Import a token
If token is on the Wyre lists it should display a "Buy" button
Once you click on the provider's "Continue to" button, a window on said provider should open, and the token to buy should already be preselected

Pre-Merge Checklist

  • PR template is filled out
  • IF this PR fixes a bug, a test that would have caught the bug has been added
  • PR is linked to the appropriate GitHub issue
  • PR has been added to the appropriate release Milestone

+ If there are functional changes:

  • Manual testing complete & passed
  • "Extension QA Board" label has been applied

@nikoferro nikoferro requested a review from a team as a code owner September 27, 2022 10:13
@github-actions
Copy link
Contributor

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.

@nikoferro
Copy link
Member Author

nikoferro commented Sep 27, 2022

@brad-decker tagging you once again since its part of the same batch of changes you have been reviewing

@darkwing darkwing merged commit 18ca016 into MetaMask:develop Sep 28, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Sep 28, 2022
@darkwing
Copy link
Contributor

Thank you @nikoferro !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants