Scaffold out Account Syncing #474

2024-02-23T00:12:18+08:00

zachgoll commented

2024-02-23 00:12:18 +08:00

(Migrated from github.com)

A bit rough around the edges here (several refactors to be done), but builds out the overall sync process for accounts.

Things that are still a TODO for future PRs:

Turbo streams need some work here to keep the sync notification, graph, and valuations list all talking with each other
The logic to build time series data in account.rb (i.e. balance_series and valuation_series) likely need to be extracted into some sort of dedicated model class, but held off to avoid early abstractions
Overall sync process will require a refactor once the concept of Transaction is introduced

A bit rough around the edges here (several refactors to be done), but builds out the overall sync process for accounts. Things that are still a TODO for future PRs: - Turbo streams need some work here to keep the sync notification, graph, and valuations list all talking with each other - The logic to build time series data in `account.rb` (i.e. `balance_series` and `valuation_series`) likely need to be extracted into some sort of dedicated model class, but held off to avoid early abstractions - Overall sync process will require a refactor once the concept of `Transaction` is introduced

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gavin/maybe#474