-
Notifications
You must be signed in to change notification settings - Fork 31
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] import bittytax exports #55
Comments
I already thought about a general input file, and why shouldn't we use this one. Great idea! We should keep in mind, that we currently do not support all of these exchanges for price gathering. #16 is tackling this issue. |
I am currently implementing a custom import format which is based in bittytax's format (see #127). I haven't checked out the bittytax conv tool, so my column format (e.g. format of datetime) might be different. Also my values (e.g. If anyone wants to work with bittytax, feel free to checkout the custom account statements type in book.py and give me a hint, if something isn't working as expected or open a PR yourself. :) |
Hi, |
Your header is currently unknown to CoinTaxman. For my implementation, I varied it a bit. Please checkout
If all prices are present in the csv, there shouldn't be a need to fetch further prices (except for the unrealized sell at end of year, but this is optional and only raises a warning, not an error). If you're willing to share the corresponding transactions from your account statement, I might be able to indicate the problem. It's easiest for me, when you create me minimal working example with a mini account statement file which shows your problem. |
Hi, thx with the newest branch I got the custom import to work for few examples. Regarding Airdrops/(Staking)Rewards I used either Type Income in BittyTax or Trade with Sell Quantity 0, so e.g. for UNI rewards: Gain/Buy X amount of UNI for 0 EUR. Both fail with assertion Error for SellQuantity, which of course kind of makes sense. In germany as far I understand it and what my tax advisor is saying, A UNI reward (or airdrop) would be Income (Sonstige Einkünfte) with e.g 100 UNI at 2 EUR, so 200 EURO Income and a trade for 200 EUR to 100 UNI the day you claim it. ( I know there is conflicting infos about that on the internet, and maybe LP rewards and airdrops would be treated differently). So I would add that trade to cointaxman and keep a seperate sheet for incomes? Or how would you treat go about, integrating airdrop/rewards into cointaxman? And what about coins which didn't have a price when you got them or claim them. To add them with price Zero kind of makes sense to me. (Kostenbasis 0) |
As far as I know. Airdrops can be taxed as (a) Einkünfte aus sonstigen Leistungen or as (b) Schenkung. There is currently only a bool in the config file which makes that all airdrops will be evaluated as (a) or all as (b). Something in between is currently not implemented (#115 ) When no price is given. It should be fine to set it to 0. Regarding your account statement. You could use the custom account statement file (see wiki) and add your airdrops there. |
Ok thx I would go with (a), but how to import an airdrop via custom import? |
Take the file from wiki. Add a row with type airdrop to the file. |
Hi, |
this shouldnt happen for airdrop either. Feel free to open a pull request. |
Should be fixed now. |
As this bittytax import / custom import is kindof implemented, I am going to close this issue. |
Hi folks,
The project bittytax (https://github.com/BittyTax/BittyTax) is a tax solution for UK.
One of its modules (bittytax_conv) allows the user to simplify their exported transaction files to a single xlsx file with different spreadsheets for every transaction file.
The columns which are the same for every file are:
Type | Buy Quantity | Buy Asset | Buy Value | Sell Quantity | Sell Asset | Sell Value | Fee Quantity | Fee Asset | Fee Value | Wallet | Timestamp
Maybe this exported file can be added as import file for CoinTaxman. This way, many exchanges could be added without the need for a new parser.
Also it would be a nice input file for custom transactions to be added.
Thank you
Sincerely
pardofelis
The text was updated successfully, but these errors were encountered: