Skip to content

Conversation

@phoenixy1
Copy link
Contributor

I ended up removing /accounts/balance/get and then later putting it back, and including both ways of calling balance in the quickstart. I think hopefully it's clear.

This PR was vibe-coded with my good friend Claude.

Contains a few fixes:

  • Updates Plaid client libraries to latest versions
  • Updates dependencies to resolve security warnings
  • Other misc fixes to resolve dependency-related warnings
  • Updates Signal API call to use a ruleset key and show ruleset evaluation results, as well as replacing some less-relevant /deprecated results like risk tier with results that are more relevant to Balance, like...balance
  • Updates .env.example file to provide better instructions and be more appropriate for Fuse
  • Fixes bug in which calling /signal/evaluate twice would result in a duplicate transaction error because the transaction_id was the same every time

ToddKerpelman
ToddKerpelman previously approved these changes Oct 31, 2025
Copy link
Contributor

@ToddKerpelman ToddKerpelman left a comment

Choose a reason for hiding this comment

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

Approved with a suggestion!

};

export const transformBalanceData = (data: AccountsGetResponse) => {
export const transformBalanceData = (data: any) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Any? Boo...

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay, I fixed this in a commit

Co-authored-by: Todd Kerpelman <[email protected]>
@phoenixy1 phoenixy1 merged commit bd9989b into master Nov 3, 2025
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