Skip to content

bug: UpdateAccountInfo does not update balances to zero due to account.Process behavior #1831

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.

Sign up for GitHub

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

Jump to bottom
Closed
junnplus opened this issue Mar 8, 2025 · 0 comments · Fixed by #1846
Closed

bug: UpdateAccountInfo does not update balances to zero due to account.Process behavior #1831

junnplus opened this issue Mar 8, 2025 · 0 comments · Fixed by #1846
Assignees
@junnplus
Labels
in progress

Comments

@junnplus
Copy link
Contributor

junnplus commented Mar 8, 2025

UpdateAccountInfo retrieves the full list of balances for an asset type. However, when updating the balances, it uses account.Process, which only processes the currencies present in the response. If an exchange does not return currencies with a zero balance, account.Process does not update them, leaving outdated balance data instead of setting them to zero.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@junnplus junnplus

Labels
in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

account: refactor for save holdings junnplus/gocryptotrader
2 participants
@thrasher- @junnplus