hodgerpodger / staketaxcsv

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

Terra Classic (LUNC, Terra 1.0, columbus) support fails to correctly calculate WBNB reward amounts. #358

Closed p00595 closed 6 months ago

p00595 commented 6 months ago

Terra Classic (LUNC, Terra 1.0, columbus) support was previously incorrectly calculating LUNC and USTC reward amounts. As of today that problem has been fixed by stake.tax development support, however the fix seems to have introduced a new bug in which the generated CSV is reporting WBNB reward amounts worth millions of dollars which is not correct as the actual amounts should only be worth a few cents at most. Could you please look into resolving this new problem.

p00595 commented 6 months ago

My wallet address is: terra17h2a42sf7kv0mrzqce90lhjvlrxtch5vw5hps4

And attached is a copy koinly-20240604-022420-lunc.csv of the CSV generated by stake.tax showing the excessively large WBNB staking rewards.

hodgerpodger commented 6 months ago

Fixed and deployed.

Thanks for the report and info.

p00595 commented 6 months ago

Thank you once again for a very prompt fix. I have not checked the results with fine toothed comb, but it appears to be calculating reward amounts correctly now.