Add Type "fork" as possibility of getting a Coin

[tobias-p]

in Coinbase, I got some BCH a while ago which was because of the fork of BCH from BTC.

This was already mentioned in my former issue (#130), but in my opinion it would be better to show in the report, that I've got those coins not because of buying or transferring from somewhere.

[jhoogstraat]

I am currently looking into the Coinbase and Coinbase Pro statements.
How does a fork look like in the statement?

[tobias-p]

Sadly, forks are not included in the statement from Coinbase... It's only known for me, since I've got that back then... Don't know, if it is mentioned in Coinbase Pro.

Nonetheless, the "fork" type would be great for correct representation of the accounts...

[provinzio]

In that case. Its enough to implement a fork operation, so that it can be used with the custom import.

For that we need to (just an idea of mine. Feel free to implement different :) )

• add a new operation type ForkGive and ForkGet (like sell and buy)
• add the matching for the fork (like sell and buy)
• add a report entry type for the fork. This requires different labels because it's more like a trade. Might be better to implement a base class Abstract tax report entry which is inherited by the current TaxReportEntry and a new TaxTradeReportEntry and inherit this by ForkReportEntry (and later on TradeReportEntry; idea of mine independent of this issue)
• add it to the tax evaluation function
• extend report export function so that the new TaxReportType is also Handled

It'll be automatically added to the XLSX report.