Skip to content
This repository was archived by the owner on Jul 27, 2025. It is now read-only.

Conversation

@Joelute
Copy link
Contributor

@Joelute Joelute commented Apr 22, 2025

This PR features changes to remove the initial_balance field allowing loan accounts to operate using its first entry as the source of truth.

Note: I've tested this in manually, however I couldn't fully verify it with Plaid

Fixes part of #2115.

@zachgoll
Copy link
Collaborator

zachgoll commented Apr 23, 2025

Hey @Joelute, thanks for the fix here, but I think we're going to need a full resolution of #2115 in order to merge. I think your original solution of adding the original balance to the Loan table made sense and we should keep it. We'll just need to integrate that domain logic with our Plaid accounts.

You'll need to get Plaid Sandbox setup to develop on that locally, which Plaid has some docs for here:

https://plaid.com/docs/sandbox/

@zachgoll
Copy link
Collaborator

@Joelute I'm going to close this one out for now. We've got some pretty big changes in #2267 around the Plaid sync and one of the changes I'm working on is dealing with the initial loan balance so I'll knock this one out!

@zachgoll zachgoll closed this May 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants