Skip to content

feat(react): add useWalletBalance query hook - #156

Merged
Cjay-Cyber-2 merged 2 commits into
ASTROIDX556:mainfrom
codexhange:feat/use-wallet-balance
Sep 1, 2026
Merged

feat(react): add useWalletBalance query hook#156
Cjay-Cyber-2 merged 2 commits into
ASTROIDX556:mainfrom
codexhange:feat/use-wallet-balance

Conversation

@codexhange

Copy link
Copy Markdown
Contributor

Adds a TanStack Query hook for a wallet's live on-chain balance, disabled when walletId is empty with a 15s stale-time default.

Closes #129

Adds a TanStack Query hook for reading a wallet's live on-chain balance,
disabled when walletId is empty with a 15s stale-time default.

Closes ASTROIDX556#129

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@codexhange Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@mergekeeper

mergekeeper Bot commented Aug 29, 2026

Copy link
Copy Markdown

Needs changes

The PR implements useWalletBalance in packages/react/src/index.ts but misses the required hook unit tests requested in the acceptance criteria.

  • packages/react/src/index.ts: Acceptance criteria require adding hook tests using React testing utilities and mock clients, which are missing from this pull request.

Reviewed commit: 0c100b6a841c22c0da701e772f09096cea1a6620.

@mergekeeper mergekeeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs changes

The PR implements useWalletBalance in packages/react/src/index.ts but misses the required hook unit tests requested in the acceptance criteria.

  • packages/react/src/index.ts: Acceptance criteria require adding hook tests using React testing utilities and mock clients, which are missing from this pull request.

Reviewed commit: 0c100b6a841c22c0da701e772f09096cea1a6620.

@Cjay-Cyber-2
Cjay-Cyber-2 merged commit d7708f9 into ASTROIDX556:main Sep 1, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement React query hook factory for wallet balance subscriptions

2 participants