-
Notifications
You must be signed in to change notification settings - Fork 69
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
Request: add balances at exchange history page #75
Comments
This is a good idea for a future addition, but requires a little too much work for now. For you wallet balance for example, you get the current wallet balance and then walk back through all transactions (transfers) that happend since that date. This should sum up to the right total, but some tokens use minting/burning/wrapping features that might not show up as a transfer. With the current setup this would be incredibly slow, to make it of any use it would probably require and additional database on a server somewhere. Again, a good idea to speed this site up, but requires quite a bit of extra work. Luckily etherscan has already done some of this work for you, just not for deposited balances. They are just a little hard to find on their homepage. Etherscan historical wallet balances:
Note that etherscan's trackers will often report a bad balance for WETH after you unwrap it, and will only correct your balance once you get some new WETH. |
Thanks for your reply. The main purpose for me is the deposited balances (on the etherdelta2 smartcontract) so the Etherscan solution is of little use to me. I understand that this is a bridge too far for now, hopefully somewhere in he future, you have my full support ;) |
The Exchange History page is working great and fast as well, kudos to that :)
However, for option " funds" it only shows me all incoming and outgoing funds
and deposits/withdrawals. Although this is great I really miss the possibility to show all historical balances as well (for both the wallet as on the smartcontract)
I was wondering if this can be added in the future, it is also a much needed option for tax purposes. Thanks in advance.
The text was updated successfully, but these errors were encountered: