-
Notifications
You must be signed in to change notification settings - Fork 16
fix(balances): use live balances on the Portfolio screen #514
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
Conversation
8f8e978 to
7761027
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The actual change is line 53.
The rest is just the eslint formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing reduce spread anti-pattern
https://prateeksurana.me/blog/why-using-object-spread-with-reduce-bad-idea/
7761027 to
9d7dc5a
Compare
9d7dc5a to
e9cb224
Compare
e9cb224 to
530bec3
Compare
c69e5e3
530bec3 to
c69e5e3
Compare
Description
https://ava-labs.atlassian.net/browse/CP-12440
Changes
Testing
Screenshots:
https://github.com/user-attachments/assets/f9c3548b-70d5-49ea-aec2-ce2b5d3436de
https://github.com/user-attachments/assets/ce44fbbd-89bd-4e47-8c71-a869467440f0
Checklist for the author
Tick each of them when done or if not applicable.