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

on refresh, balance header shows incorrect numbers #2303

Closed
jbibla opened this issue Mar 16, 2019 · 4 comments · Fixed by #2322
Closed

on refresh, balance header shows incorrect numbers #2303

jbibla opened this issue Mar 16, 2019 · 4 comments · Fixed by #2322
Assignees
@faboweb
Labels
bug 🐛 issues related to unhandled errors in the code that need to be fixed

Comments

@jbibla
Copy link
Collaborator

jbibla commented Mar 16, 2019 •
edited
Loading

Describe the bug

  • total atom balance calculation is off
  • no delegations card appears even though there are delegations

To Reproduce

  • log in with ledger
  • refresh the page
  • see total atom balance displayed with same number as "available"
  • "no delegations" card appears even though there are delegations

Expected behavior

  • i expect to see the correct atom total
  • expect to see my delegations

Additional context

  • re-rendering the page resets the data to be correct
@jbibla jbibla added the bug 🐛 issues related to unhandled errors in the code that need to be fixed label Mar 16, 2019
@faboweb
Copy link
Collaborator

faboweb commented Mar 20, 2019

Seems fixed on develop

@jbibla
Copy link
Collaborator Author

jbibla commented Mar 20, 2019

not exactly. still showing 0.0000 for a second or two and then available balances for both total and available and then it updates to the correct balance...

@faboweb
Copy link
Collaborator

faboweb commented Mar 20, 2019

So should we only show the balance once wallet and delegation have been loaded?

@jbibla
Copy link
Collaborator Author

jbibla commented Mar 20, 2019

yeah, should be -- before 0.

@faboweb faboweb self-assigned this Mar 20, 2019
@faboweb faboweb mentioned this issue Mar 20, 2019
Merged
5 tasks
@faboweb faboweb mentioned this issue Mar 21, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@faboweb faboweb

Labels
bug 🐛 issues related to unhandled errors in the code that need to be fixed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fabo/2303 balance display as whole luniehq/lunie
2 participants
@faboweb @jbibla