beancount_reds_importers icon indicating copy to clipboard operation
beancount_reds_importers copied to clipboard

Published 20 hours ago • verified publisher icon redstreet

add TD Ameritrade CSV importer

Open reedlaw opened this issue 1 year ago • 4 comments

After getting some help with this issue, I think this importer is good enough to share.

reedlaw avatar Mar 28 '23 01:03 reedlaw

Great, thanks for sharing! Left some comments. Also, would you be able to share an anonymized .csv file that can be used for testing?

redstreet avatar Mar 28 '23 02:03 redstreet

I've added an example and script following the Fidelity CSV pattern. I wasn't able to run it because I'm not sure how to require the local Python package instead of the system version. I think it involves creating a virtual environment but I'm not proficient enough in Python to know how to do it.

reedlaw avatar Mar 28 '23 11:03 reedlaw

That's great. The example is sufficient. I'll add the test code to run it, which I need to unify for all importers anyway.

If you don't mind addressing the other comments above, I'll be happy to merge in this PR. Thanks!

redstreet avatar Apr 13 '23 23:04 redstreet

Hi @reedlaw, just checking to see if you are still inclined to complete this PR and merge? Thanks!

redstreet avatar Oct 29 '23 02:10 redstreet