pyFIFOtax icon indicating copy to clipboard operation
pyFIFOtax copied to clipboard
verified publisher icon stadlmax

Converter modifications and additions

Open szotsaki opened this issue 1 year ago • 4 comments

  • Modify the converters to have one calling place (converter.py)
  • Add DEGIRO import
  • Transactions spreadsheet output according the new format

Draft until the original PR is merged since more testing is needed.

Please, merge it normally, without squashing.

szotsaki avatar Jun 24 '24 09:06 szotsaki

I trust you with the correctness of the DeGiro and IBKR converts, can't say much about correctness besides most of the things making sense.

stadlmax avatar Aug 18 '24 22:08 stadlmax

Is this the latest code? First time user, trying to make sense of this program and repo.

aaptel avatar May 19 '25 21:05 aaptel

Is this the latest code? First time user, trying to make sense of this program and repo.

Yes, this is the latest code. However, in its current form it doesn't work as it relies on future YFinance fixes. Please, look out for the modifications in https://github.com/stadlmax/pyFIFOtax/pull/6 in a few days. I'll then rebase this PR on top if and it should be good to use.

szotsaki avatar May 25 '25 19:05 szotsaki

I ended up using the XL sheet someone linked in the Google Docs and did it manually. Hopefully next time I'll use this tool! Thanks

aaptel avatar May 26 '25 08:05 aaptel