Skip to content

Conversation

@jvsena42
Copy link
Member

@jvsena42 jvsena42 commented Aug 4, 2025

Related to #268

Description

This PR checks if the transfer amount is at least enough to pay the blockchain and LSP fees and display an error message

Preview

QA Notes

  • Set a small amount to transfer -> It should display an error message with the minimum amount and override the current amount

@jvsena42 jvsena42 self-assigned this Aug 4, 2025
@jvsena42 jvsena42 marked this pull request as ready for review August 4, 2025 12:12
@jvsena42 jvsena42 requested a review from ovitrif August 4, 2025 12:18
@jvsena42 jvsena42 enabled auto-merge August 4, 2025 12:18
Copy link
Collaborator

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

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

Have one remark, otherwise LGTM!

@jvsena42 jvsena42 requested a review from ovitrif August 4, 2025 14:02
@jvsena42 jvsena42 merged commit 43d847f into master Aug 4, 2025
5 checks passed
@jvsena42 jvsena42 deleted the fix/min-transfer-validation branch August 4, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants