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
[VDG] Fix Amount Paste #12661
[VDG] Fix Amount Paste #12661
Conversation
ichthus1604
commented
Mar 13, 2024
•
edited
edited
- Removes the restriction to paste amounts larger than current wallet balance
- Removes the automatic replacement that occurs when trying to paste amounts larger than total BTC in existence. This rule doesn't make sense for clipboard.
- Validations continue to work for both cases in the Send dialog: user won't be able to hit Continue button if input is invalid.
- Paste suggestions (flyout) keep their existing behavior: they will only appear if you have enough funds and there's a valid value to paste in the box.
- Pasting any kind of nonsense (non numeric characters, etc) is disallowed.
- Fixes [UI] Allow more than balance amount paste #12645
@ichthus1604 Please fix the tests too |
After #12660 is merged, I need to fix this: #12660 (comment) in this PR. Since this PR touches flyout related code, it will be less problematic to include this fix in here rather than in the other PR. |
Merged. |
Done ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pasting amount with a different decimal separator doesn't work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested LGTM