FatStx / FatStx.github.io

FatStx Site
8 stars 2 forks source link

feature request: add wallet balance #4

Closed dantrevino closed 2 years ago

dantrevino commented 2 years ago

Assume this would be something like:

egtalbot commented 2 years ago

Adding wallet balance would helpful in looking at this as a general transaction report but it doesn't feel like a core requirement for a tax report/upload.

That said, it's worth looking at longer term some things to consider: -wallets can have a lot of different coins and NFTs, so it doesn't lend itself to adding to either the screen or the csv -I don't think it would be a good approach to execute an API call for each block height with a transaction. Could slow things down a lot. But there may be some other calls we can find.

dantrevino commented 2 years ago

Agree with the api call issue, but this was requested by my accountant, so disagree that its not a good fit for a tax report. Thanks for considering it either way.

egtalbot commented 2 years ago

Interesting - your accountant should not need wallet balance for taxes on STX transactions from what I can tell. Maybe other countries have different rules. That said, I don't disagree that it would be nice to have and I would add it soon if it were simple.

I think before we got to that, we'd need to add functionality to choose what columns to show, so if someone wanted to take the performance hit they could choose to do so. Selecting columns to show and exclude is something we do have on a radar to do.

Another option is to create a separate report for wallet balances. That's probably not what your accountant is looking for though.

dantrevino commented 2 years ago

fwiw, I believe its more of a desire for validation.