-
Notifications
You must be signed in to change notification settings - Fork 38
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
fix: dont use sapi for displaying balances #1744
fix: dont use sapi for displaying balances #1744
Conversation
0xKheops
commented
Dec 11, 2024
•
edited
Loading
edited
- Changed native balance module to include one value per validator, in case the account is staking on multiple validators.
- Adjusted portfolio components to read hotkey from the balance rows instead of fetching them from chain
🦋 Changeset(s) detectedThis PR includes changeset(s) for the following changed packages:
Not sure what this means? Click here to learn what changesets are. |
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.
Amazing! ❤️
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.
LGTM!
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.
LGTM!