-
Notifications
You must be signed in to change notification settings - Fork 31
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
Transfers between Coinbase and Coinbase Pro not considered #138
Comments
When the information if the withdrawal/deposit is missing think the statement files we have to count them as the same wallet... |
If that is allowed by our tax laws. They are technically two separate wallets. |
I cannot confirm that, but if we are unable to reconstruct the deposit/withdraw between coinbase and coinbase pro, we have no choice. Maybe check that again or write an email to the Coinbase support? |
Their support site says
I mailed then still.. lets see. Another thing: |
But it doesn't have do be deposit+trade? Or is it always deposit+trade? In that case it's possible and necessary |
Today, you actually can. I am unsure whether you could always deposit fiat. I am unsure how to distinguish buys with deposited money or direct buy. |
Regarding the transfers between coinbase and coinbase pro:
Coinbase and coinbase pro are separeted wallets. See https://help.coinbase.com/en/pro/managing-my-account/funding-your-account/how-to-transfer-funds-between-your-coinbase-pro-and-coinbase-accounts
You mentioned, that you found a file which includes the deposits between these to. Please implement it or raise an error when it's missing, but the user selected |
The reports from Coinbase and Coinbase Pro do not explicitly state transfers between the two.
In case of
MULTI_DEPOT
being enabled, this leads to missing coins on either side.I am unsure whether the two can be considered as distinct exchanges at all.
This hints at that they are not.
Edit:
Just saw that the
Account
statement on Coinbase Pro does include deposits and withdrawals. I think we should use that instead of theFills
statement. Doesn't fix the issue on the Coinbase side though.The text was updated successfully, but these errors were encountered: