-
Notifications
You must be signed in to change notification settings - Fork 7
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
Consensus dashboard account list card #1160
Conversation
buberdds
commented
Jan 18, 2024
•
edited
Loading
edited
Deployed to Cloudflare Pages
|
9d4f66e
to
870d546
Compare
src/oasis-nexus/api.ts
Outdated
// TODO: remove when API returns total, looking at query filters we store that data in Nexus | ||
const total = BigInt(account.available) + BigInt(account.escrow) + BigInt(account.debonding) |
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.
This calculation is wrong. API does not return the correct fields, and debonding is named misleadingly. History of wrong math: oasisprotocol/nexus#84 and oasisprotocol/oasis-wallet-web#927
https://nexus.oasis.io/v1/consensus/accounts?limit=1
"available":"0","debonding":"0","escrow":"0"
single account API returns correct fields
https://nexus.oasis.io/v1/consensus/accounts/oasis1qz78ap0456g2rk7j6rmtvasc9v2kjhz2s58qgj90
"available":"21454060793","debonding":"0","debonding_delegations_balance":"0","delegations_balance":"103544825467","escrow":"0"
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.
I will leave TODOs in this case.
d2c3637
to
a858a1c
Compare
a858a1c
to
02667f8
Compare
02667f8
to
7677996
Compare