Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: YNAB4 import leads to malformed split transactions #3561

Open
2 tasks done
mullermn opened this issue Oct 4, 2024 · 3 comments
Open
2 tasks done

[Bug]: YNAB4 import leads to malformed split transactions #3561

mullermn opened this issue Oct 4, 2024 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed importers Related to importers

Comments

@mullermn
Copy link

mullermn commented Oct 4, 2024

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

After migrating to Actual from a large YNAB4 install I found that my split transactions did not show up correctly:
image

There is evidence that the data was imported behind the scenes but is not displayed:
image

I suspect this is because YNAB4 [can] tracks payees for split transactions solely at parent transaction level, and does not require payees for the splits, while Actual computes the parent payee name from that of the splits?

This doesn't break the app but introduces the dangerous gotcha that these transactions will not show up in reports and filters based on Payee name.

Split transfers do not seem to be affected:
image

The repair split transactions tool seems to handle this correctly, but ideally the situation should not exist in the first place:
image

Where are you hosting Actual?

Pikapods

What browsers are you seeing the problem on?

Desktop App (Electron)

Operating System

Mac OSX

@mullermn mullermn added the bug Something isn't working label Oct 4, 2024
@MatissJanis
Copy link
Member

👋 Hi. Could you upload a sample ynab export that we could use to reproduce the issue? Thanks

@MatissJanis MatissJanis added the needs info We need more information from the OP before continuing label Oct 4, 2024
RubenOlsen added a commit to actualbudget/docs that referenced this issue Oct 6, 2024
Added in a Known issues section for YNAB4 imports to cover an issue with
split transactions not receiving correct payee details.

Once this issue is resolved the documentation note can be removed:
actualbudget/actual#3561

---------

Co-authored-by: Ruben Olsen Lærk <[email protected]>
@aa-hr
Copy link

aa-hr commented Oct 7, 2024

My Budget~100D554C.ynab4.zip

Hi, sorry for the delay. I started by trying to prune down my real budget to produce a real world sample without too much personal data in it but that proves to be borderline impossible due to the size of it, sorry. I have made this one, which is a brand new budget and does demonstrate the visual issue with Payees not showing up on splits.
YNAB4:
image

Actual:
image

The parent transaction in this example DOES seem to show up in filters in Actual, so either I was mistaken about that or that was a consequence of something else in my real budget data (I can't check now, having run the 'fix splits' process). However, I think this visual bug would be worth fixing in its own right as it is a bit ugly and confusing.

I have also created a second transaction (after I took those screenshots) with a transfer as one of the splits. The transfer element seems to get imported correctly, so it looks like the missing behaviour is just to take the payee at parent level in YNAB and apply it to any non-transfer splits.

Edit: Sorry, I am the original reporter, just uploaded this while logged in on a work account by mistake.

@youngcw
Copy link
Member

youngcw commented Oct 8, 2024

I think this is also happening with nynab imports

@MatissJanis MatissJanis added help wanted Extra attention is needed importers Related to importers and removed needs info We need more information from the OP before continuing labels Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed importers Related to importers
Projects
None yet
Development

No branches or pull requests

4 participants