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

Read vbank assets from x/bank not wallet.current #49

Closed
Tracked by #6788
samsiegart opened this issue Jan 24, 2023 · 1 comment
Closed
Tracked by #6788

Read vbank assets from x/bank not wallet.current #49

samsiegart opened this issue Jan 24, 2023 · 1 comment
Assignees
@samsiegart

Comments

@samsiegart
Copy link
Contributor

Some (currently all except zoe invitations) purse balances can be queried from the cosmos x/bank module. Since purses are lazily created now, the UI needs to rely on querying x/bank to know the user's true asset balances. The UI should be updated to collate x/bank assets with purse balances.
@samsiegart samsiegart self-assigned this Jan 24, 2023
@dckc
Copy link
Member

dckc commented Jan 25, 2023 •
edited
Loading

fwiw, test-watch-balances.js is WIP (Agoric/agoric-sdk#6807) to read x/bank balances using casting, to get

  • proofs
  • multi-RPC-node fail-over
  • batching

and all those goodies.

p.s. for follow-up, see Agoric/agoric-sdk#6870 (comment)

This was referenced Jan 26, 2023
Merged
Merged
This was referenced Jan 27, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samsiegart samsiegart

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dckc @samsiegart