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

The sign multisignature modal doesn't re-validate after input value changes #3593

Closed
reyraa opened this issue Jun 9, 2021 · 0 comments · Fixed by #3666
Closed

The sign multisignature modal doesn't re-validate after input value changes #3593

reyraa opened this issue Jun 9, 2021 · 0 comments · Fixed by #3666
Assignees
@isalga
Labels
type: bug
Milestone
Sprint 52

Comments

@reyraa
Copy link
Contributor

reyraa commented Jun 9, 2021

Expected behavior

If I input an invalid value into the textarea, and then fix its issues or input a new valid transaction, the error message should disappear and the CTA must get activated.

Actual behavior

The CTA remains disabled and the error message doesn't disappear.

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

2.0.0-beta.4
@sridharmeganathan sridharmeganathan added this to the Sprint 52 milestone Jun 25, 2021
@reyraa reyraa assigned reyraa and isalga and unassigned reyraa and isalga Jun 25, 2021
@sridharmeganathan sridharmeganathan mentioned this issue Jun 29, 2021
Closed
@isalga isalga assigned isalga and unassigned reyraa Jun 30, 2021
@isalga isalga mentioned this issue Jun 30, 2021
Merged
UsamaHameed pushed a commit that referenced this issue Jul 1, 2021
Merge pull request #3666 from LiskHQ/3593-sign-msign-tx-revalidation
193539b 
Revalidate multisignature transaction - Closes #3593
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isalga isalga

Labels
type: bug
Projects
None yet
Milestone
Sprint 52
Development

Successfully merging a pull request may close this issue.

Revalidate multisignature transaction - Closes #3593 LiskHQ/lisk-desktop
3 participants
@reyraa @isalga @sridharmeganathan