[PR #5861] [WIP] Fix merging split transactions (#5781) #6171

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

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

State: closed
Merged: No


Fixes #5781

This bug fixes an issue where, during transaction merging, if the transaction that is dropped has split transactions and the transaction that was kept does not then the dropped transaction's splits disappear and don't get re-parented to the kept transaction.

Now, the behavior is:

  • If the dropped transaction is a split transaction and the kept transaction does not, the dropped transaction's splits are re-parented to the kept transaction
  • If both the dropped and kept transaction are split transactions the dropped transaction's splits are properly deleted and just the kept transaction's splits are kept around

This is my first time doing this, so please feel free to let me know if anything needs to be changed. Thanks!

**Original Pull Request:** https://github.com/actualbudget/actual/pull/5861 **State:** closed **Merged:** No --- Fixes #5781 This bug fixes an issue where, during transaction merging, if the transaction that is dropped has split transactions and the transaction that was kept does not then the dropped transaction's splits disappear and don't get re-parented to the kept transaction. Now, the behavior is: - If the dropped transaction is a split transaction and the kept transaction does not, the dropped transaction's splits are re-parented to the kept transaction - If both the dropped and kept transaction are split transactions the dropped transaction's splits are properly deleted and just the kept transaction's splits are kept around This is my first time doing this, so please feel free to let me know if anything needs to be changed. Thanks!
GiteaMirror added the pull-request label 2026-02-28 21:24:54 -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#6171