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

Feature request: option to store raw OFX #113

Open
emin63 opened this issue Aug 3, 2020 · 3 comments
Open

Feature request: option to store raw OFX #113

emin63 opened this issue Aug 3, 2020 · 3 comments

Comments

@emin63
Copy link
Contributor

emin63 commented Aug 3, 2020

I have a feature request: allow an option to store the raw OFX.

For example, if you do ledger-autosync --account <some_acct> --storeraw /path/to/raw.ofx then it would be nice if l-a could store the raw OFX data that it downloads in /path/to/raw.ofx. This is helpful for debugging and also helpful if l-a does something not quite right with the OFX. For example, I am still having some problems with fees and commissions. It would be nice if I could run l-a with the --storeraw flag to store the raw OFX it gets and then reprocess that OFX later when I find/fix bugs.

What do you think?

Thanks,
-Emin

@egh
Copy link
Owner

egh commented Aug 3, 2020

I think this is a great idea! ledger-autosync will sometimes make multiple requests, so there might be multiple files, and I'm not 100% sure you can access the raw, unprocessed OFX using ofxclient easily. But I think it's a great idea. Unfortunately I probably don't have the time to work on this in the near future.

@501st-alpha1
Copy link
Contributor

I agree, this would definitely be useful, and it also would save me the extra step of downloading backup OFX files after running ledger-autosync.

The multiple requests / files is fair, but I'd be fine with that.

If I'm reading the code right, the initial download is returned as a string, and it's not until a couple lines later that it's parsed into a Python object, so it should be possible to dump it to a file before that.

@egh
Copy link
Owner

egh commented Aug 4, 2020

If I'm reading the code right, the initial download is returned as a string, and it's not until a couple lines later that it's parsed into a Python object, so it should be possible to dump it to a file before that.

Your reading looks correct to me! Sorry for the misinformation; it's been a while since I worked on that cod.e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants