You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At a quick glance at the Transaction class, it is not immediately obvious whether the btc and usd attributes include or exclude the fee. It would be nice if there were a comment for this class's definition that clarifies this aspect of the API. (Based on looking at the BitcoindParser class implementation, I am reasonably confident that it excludes the fee.)
The text was updated successfully, but these errors were encountered:
JeremyRand
pushed a commit
to JeremyRand/bitcoin-taxes
that referenced
this issue
Apr 13, 2023
At a quick glance at the
Transaction
class, it is not immediately obvious whether thebtc
andusd
attributes include or exclude the fee. It would be nice if there were a comment for this class's definition that clarifies this aspect of the API. (Based on looking at theBitcoindParser
class implementation, I am reasonably confident that it excludes the fee.)The text was updated successfully, but these errors were encountered: