Skip to content
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

Support deposit/withdrawal queries for binance. #458

Closed
LefterisJP opened this issue Aug 10, 2019 · 0 comments
Closed

Support deposit/withdrawal queries for binance. #458

LefterisJP opened this issue Aug 10, 2019 · 0 comments
Labels
exchange Issues related to integration with an exchange

Comments

@LefterisJP
Copy link
Member

LefterisJP commented Aug 10, 2019

Problem Definition

Rotkehlchen so far did not support inclusion of deposit/withdrawal queries for Binance since there is no such endpoint in their API docs: https://github.com/binance-exchange/binance-official-api-docs/blob/master/rest-api.md

Task

After a twitter rant about absence of API endpoint for deposits/withdrawal queries for Binance it seems that there actually is an endpoint but in a different document:

https://github.com/binance-exchange/binance-official-api-docs/blob/master/wapi-api.md

No idea why the separate document and no reference from one to another .... but this seems to enable us to query deposits/withdrawals for binance too. So the task for this issue is:

  • Support deposits/withdrawals queries for binance and take them into account in the profit/loss calculation.
  • Write tests for:
    • The happy case of a few deposit/withdrawal queries
    • The unhappy case of unexpected data returned either from a deposit or a withdrawal query.

An important note for implementation here is that according to this twitter thread only 3 months are returned from those API endpoint at a time so may need to perform a pagination of 3 months from the time of the first trade or creation of the account.

Can't confirm it ^. With a simple query of all deposits I got all deposits I ever did in Binance 1.5 year ago.

@LefterisJP LefterisJP added the exchange Issues related to integration with an exchange label Aug 10, 2019
LefterisJP added a commit to LefterisJP/rotkehlchen that referenced this issue Aug 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exchange Issues related to integration with an exchange
Projects
None yet
Development

No branches or pull requests

1 participant