Skip to content

feat(sdk-coin-sui): update balance querying to handle fundsInAddressBalance field#8332

Open
abhishekagrawal080 wants to merge 1 commit intomasterfrom
CSHLD-407
Open

feat(sdk-coin-sui): update balance querying to handle fundsInAddressBalance field#8332
abhishekagrawal080 wants to merge 1 commit intomasterfrom
CSHLD-407

Conversation

@abhishekagrawal080
Copy link
Copy Markdown
Contributor

TICKET: CSHLD-407

@abhishekagrawal080 abhishekagrawal080 changed the title feat(sui): Update balance querying to handle fundsInAddressBalance field feat(sui): update balance querying to handle fundsInAddressBalance field Mar 23, 2026
@abhishekagrawal080 abhishekagrawal080 changed the title feat(sui): update balance querying to handle fundsInAddressBalance field feat(sdk-coin-sui): update balance querying to handle fundsInAddressBalance field Mar 23, 2026
@abhishekagrawal080 abhishekagrawal080 force-pushed the CSHLD-407 branch 3 times, most recently from d4060cc to cadeda0 Compare March 25, 2026 10:02
@veetragjain
Copy link
Copy Markdown
Contributor

@claude

@abhishekagrawal080 abhishekagrawal080 marked this pull request as ready for review March 25, 2026 15:05
@abhishekagrawal080 abhishekagrawal080 requested a review from a team as a code owner March 25, 2026 15:05
@veetragjain
Copy link
Copy Markdown
Contributor

@claude

veetragjain
veetragjain previously approved these changes Mar 31, 2026
@sachushaji
Copy link
Copy Markdown
Contributor

@claude

@lokesh-bitgo
Copy link
Copy Markdown
Contributor

@claude Please do a thorough review of this PR.

Focus especially on:

  • any bugs or incorrect logic in the changes
  • whether these changes could impact existing flows or cause regressions
  • whether the implementation is aligned with the requirement/design
  • whether the changes are correct and complete
  • missing edge cases, validation gaps, and test coverage gaps

Please call out concrete issues clearly and separate them by severity if possible.

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.

4 participants