[PR #2534] ♻️ (bank-sync) unify the sync/reconciliation logic for internal & external sync #4352

Closed
opened 2026-02-28 20:54:46 -06:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/actualbudget/actual/pull/2534

State: closed
Merged: Yes


Unifying the reconciliation and syncing logic to reduce duplication.

Previously we had reconcileTransactions, reconcileExternalTransactions, syncAccount and syncExternalAccount. Now we have only one of each.

The logic there is still not exactly clean. But at least now we have a single place for it instead of copy&paste duplication.

**Original Pull Request:** https://github.com/actualbudget/actual/pull/2534 **State:** closed **Merged:** Yes --- Unifying the reconciliation and syncing logic to reduce duplication. Previously we had `reconcileTransactions`, `reconcileExternalTransactions`, `syncAccount` and `syncExternalAccount`. Now we have only one of each. The logic there is still not exactly _clean_. But at least now we have a single place for it instead of copy&paste duplication.
GiteaMirror added the pull-request label 2026-02-28 20:54:46 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#4352