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

Error handling when no crypto currencies available on selected region #4360

Merged
merged 8 commits into from
May 23, 2022

Conversation

alaahd
Copy link
Contributor

@alaahd alaahd commented May 17, 2022

Description

This change will notify the user once cryptocurrencies are not available to purchase on specific selected region on the selected network.

For example: User selected region is "Alabama - USA" and the wallet is connected to "Optimism" network.

We still give the user an option to choose a different region.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

Screenshots/Recordings

If applicable, add screenshots or recordings to visualize the changes

Issue

Progresses #???

@alaahd alaahd requested a review from a team as a code owner May 17, 2022 18:15
@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.

@alaahd alaahd changed the title Proper error handling for no crypto currencies available on certain region Error handling when no crypto currencies available on selected region May 17, 2022
@alaahd alaahd requested a review from rickycodes May 17, 2022 20:00
Copy link
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alaahd alaahd requested a review from Cal-L May 18, 2022 07:56
@alaahd
Copy link
Contributor Author

alaahd commented May 18, 2022

I have pushed on more commit for improvement suggested by @wachunei appreciate if we can review again.

@alaahd alaahd requested a review from wachunei May 18, 2022 13:47
@alaahd alaahd requested a review from wachunei May 19, 2022 18:16
@wachunei wachunei merged commit 8cb3f1d into MetaMask:release/5.2.0 May 23, 2022
@github-actions github-actions bot locked and limited conversation to collaborators May 23, 2022
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.

4 participants