search

hodgerpodger / staketaxcsv

python repo to create blockchain CSVs
MIT License
252 stars 119 forks source link

fix - dont filter out rows for ctc format #338

Closed ProfessorX737 closed 8 months ago

ProfessorX737 commented 8 months ago

CryptoTaxCalculator was missing transactions when importing certain wallets as staketax filteres out _UKNOWN trade types. The CryptoTaxCalculator parser will ignore trades types that aren't valid or treat in/out tx types as uncategorised transactions. This pr should give CTC a more complete picture of the trade history

Tested with this wallet: inj1engv8j6lgepvzqfk62jths8str6s424lcjehad on INJ

hodgerpodger commented 8 months ago

Thanks @ProfessorX737 ! Only made a slight modification (with same functional behavior).

Pushed and deployed with https://github.com/hodgerpodger/staketaxcsv/commit/fab5484ad1943cec944179bbf2c381defbc8afe0 today.

ProfessorX737 commented 8 months ago

Thanks @hodgerpodger! Much appreciated