-
Notifications
You must be signed in to change notification settings - Fork 68
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
Filter unrecognized from extra URI parameters IAW BIP21 #935
Conversation
Converted to draft pending more testing. Or we can merge and test w/ testers :) |
We should expand on the parse util to handle various types of payment uris (bitcoin based, monero based, etc). The PR currently only affects the main desktop send screen. Nothing on mobile and possibly other send screens on desktop will not work with this either at the moment. |
bitcoin URIs now also validate properly
Re:
This issue has been resolved by b73628d |
instead of duplicating logic
Closes #567
Tasks:
TODO refer to BIP21 to see if any other params are required