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.

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

Wallet page throws a state update warning #4029

Closed
Tracked by #3963
eniolam1000752 opened this issue Dec 30, 2021 · 1 comment · Fixed by #4030
Closed
Tracked by #3963

Wallet page throws a state update warning #4029

eniolam1000752 opened this issue Dec 30, 2021 · 1 comment · Fixed by #4030
Assignees
Milestone

Comments

@eniolam1000752
Copy link
Contributor

eniolam1000752 commented Dec 30, 2021

Expected behavior

On the first entry to the wallet's page, there should be no warning stating Cannot update during an existing state transition (such as within 'render'). Render methods should be a pure function of props and state.

Actual behavior

On the first entry to the wallet's page, there is a warning stating Cannot update during an existing state transition (such as within 'render'). Render methods should be a pure function of props and state.

image

Steps to reproduce

  • Login on the lisk desktop application using the web
  • Visit the wallet page
  • On the browser's console, It would be seen that the warning would be thrown.

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

all environment.

@welcome
Copy link

welcome bot commented Dec 30, 2021

Thanks for opening your first issue here! Be sure to follow the issue template for bug report or feature request!

@eniolam1000752 eniolam1000752 self-assigned this Dec 31, 2021
@ManuGowda ManuGowda added this to the Spike 3 milestone Dec 31, 2021
@ManuGowda ManuGowda linked a pull request Dec 31, 2021 that will close this issue
@ManuGowda ManuGowda modified the milestones: Spike 3, Sprint 65 Jan 1, 2022
ManuGowda added a commit that referenced this issue Jan 3, 2022
…update-warning

Fix state update warning on wallet page - Closes #4029
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants