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]: QFX Import Broken on new parser #2331

Closed
1 of 5 tasks
matthewkdies opened this issue Feb 4, 2024 · 7 comments
Closed
1 of 5 tasks

[Bug]: QFX Import Broken on new parser #2331

matthewkdies opened this issue Feb 4, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@matthewkdies
Copy link

Verified issue does not already exist?

  • I have searched and found no existing issue

Is this related to GoCardless, Simplefin or another bank-sync provider?

  • I have checked my server logs and could not see any errors there
  • I will be attaching my server logs to this issue
  • I will be attaching my client-side (browser) logs to this issue
  • I understand that this issue will be automatically closed if insufficient information is provided

What happened?

When importing QFX files from my bank (PNC), the import fails with no logs. Previously, I would uncheck the "Use experimental parser" option in the settings to fix this, but this option has disappeared in the current release. I also tried downgrading to 24.1.0, though the migration failed and the app wouldn't start.

If desired, I can include the relevant QFX file (censored) in the comments.

What error did you receive?

image

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Firefox, Safari

Operating System

Mac OSX

@matthewkdies matthewkdies added the bug Something isn't working label Feb 4, 2024
@youngcw
Copy link
Member

youngcw commented Feb 4, 2024

Please include a sample qfx. We cant do anything useful without it

@matthewkdies
Copy link
Author

Please include a sample qfx. We cant do anything useful without it

Had to change the extension to .txt as .QFX isn't supported by GitHub. I've modified the memos, names, and amounts but the rest is as is from PNC.

quickenExport(1).txt

@youngcw
Copy link
Member

youngcw commented Feb 5, 2024

ping @joel-jeremy

@Oglaf
Copy link

Oglaf commented Feb 6, 2024

I'm also having problems importing OFX

@twk3
Copy link
Contributor

twk3 commented Feb 8, 2024

I have a fix for the error you ran into in #2342

But @matthewkdies it appears that the OFX file you are trying to import is for multiple bank accounts. (Though only 1 has transactions). Actuals current transaction importers are meant to be per-account. Does your bank let you download per-account qfx files?

@matthewkdies
Copy link
Author

matthewkdies commented Feb 8, 2024

Unfortunately, no. PNC's QFX/OFX export is terrible IMO -- it doesn't let you filter by date selection for import, so I have to do that myself with a script I wrote/am writing. It also exports for all accounts, and PNC's vast majority of users have three: spend, savings, and reserve.

Edit: FWIW, I believe that PNC exports the OFX file based on the selected account and adds the appropriate transactions. So when I am in "reserve" those transactions do appear.

@twk3
Copy link
Contributor

twk3 commented Feb 8, 2024

The parse error is fixed in master with #2342

@twk3 twk3 closed this as completed Feb 8, 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
Projects
None yet
Development

No branches or pull requests

4 participants