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

Failed to validate minimum fee requirement for delegate registration #4053

Closed
Tracked by #3963
ManuGowda opened this issue Jan 10, 2022 · 0 comments · Fixed by #4064
Closed
Tracked by #3963

Failed to validate minimum fee requirement for delegate registration #4053

ManuGowda opened this issue Jan 10, 2022 · 0 comments · Fixed by #4064
Assignees
@isalga
Labels
type: bug unplanned
Milestone
Sprint 65

Comments

@ManuGowda
Copy link
Contributor

Expected behavior

Should validate delegate registration transaction for minimum fee requirement and show correct error message for fee requirement

Actual behavior

Failed to validate delegate registration transaction for minimum fee requirement

Screenshot 2022-01-10 at 11 44 33

Steps to reproduce

  • Have account available balance less than 10 LSK
  • Register delegate

Which version(s) does this affect? (Environment, OS, etc...)

2.2.0
@ManuGowda ManuGowda added this to the Sprint 65 milestone Jan 10, 2022
@isalga isalga self-assigned this Jan 11, 2022
@isalga isalga mentioned this issue Jan 11, 2022
Merged
isalga pushed a commit that referenced this issue Jan 11, 2022
@ManuGowda
Validate fee at register delegate form - Closes #4053 (#4064)
c6aefd3 
* Add disable button condition

* Fix hasUserEnoughFunds calculation

* Change error message

* Add unit test

* Remove wrapper.update

Co-authored-by: Manu <[email protected]>
@ManuGowda ManuGowda mentioned this issue Jan 18, 2022
Closed
37 tasks
reyraa pushed a commit that referenced this issue Jan 24, 2022
@ManuGowda @reyraa
Validate fee at register delegate form - Closes #4053 (#4064)
79354c5 
* Add disable button condition

* Fix hasUserEnoughFunds calculation

* Change error message

* Add unit test

* Remove wrapper.update

Co-authored-by: Manu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isalga isalga

Labels
type: bug unplanned
Projects
None yet
Milestone
Sprint 65
Development

Successfully merging a pull request may close this issue.

Validate fee at register delegate form - Closes #4053 LiskHQ/lisk-desktop
3 participants
@reyraa @isalga @ManuGowda