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.

Sign up for GitHub

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

Jump to bottom

XMR addresses should only save when they are valid XMR addresses #23

Open
derrickhorton opened this issue Mar 21, 2023 · 2 comments
Open

XMR addresses should only save when they are valid XMR addresses #23

derrickhorton opened this issue Mar 21, 2023 · 2 comments
Assignees
@pshenmic
Labels
enhancement New feature or request

Comments

@derrickhorton
Copy link
Contributor

derrickhorton commented Mar 21, 2023 •
edited
Loading

Right now, the app checks the addresses that the user inputs for many coins, but not for XMR.

  • When the user taps "Paste," the app allows the user to paste in anything, valid XMR address or otherwise!

  • When the user taps "Scan," the app allows the user to scan any QR code, valid XMR address or otherwise!

This enhancement would enable the user to receive a helpful error message when he scans the wrong thing:

  • "Invalid XMR address"
@derrickhorton derrickhorton added the enhancement New feature or request label Mar 21, 2023
@powco-dev
Copy link

powco-dev bot commented Mar 21, 2023

Thank you for making this issue!

Please get it some attention by boosting it here https://powco.dev/github/anypay/merchant-app/issues/23

Bounties may be sent to this address https://whatsonchain.com/address/1FyN9v13Q8vyULXMEp9sLAVXPeG8VjwPXU and will be paid to the worker who completes your issue.

1 similar comment
@powco-dev
Copy link

powco-dev bot commented Mar 21, 2023

Thank you for making this issue!

Please get it some attention by boosting it here https://powco.dev/github/anypay/merchant-app/issues/23

Bounties may be sent to this address https://whatsonchain.com/address/1FyN9v13Q8vyULXMEp9sLAVXPeG8VjwPXU and will be paid to the worker who completes your issue.

@pshenmic pshenmic mentioned this issue Aug 15, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pshenmic pshenmic

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@derrickhorton @pshenmic