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

Fixed setting Order.AveragePrice to null when Quantity is zero #646

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

vslee
Copy link
Collaborator

@vslee vslee commented Sep 14, 2021

  • also fixed Binance statii: PendingCancel and Error
  • also fixed fetching only Open orders on Coinbase OnGetOpenOrderDetailsAsync()
  • made Binance userdata properties public

- also fixed Binance statii: PendingCancel and Error
- also fixed fetching only Open orders on Coinbase OnGetOpenOrderDetailsAsync()
@vslee vslee added the bug label Sep 14, 2021
@vslee vslee requested a review from jjxtra September 15, 2021 00:04
@vslee vslee merged commit 7fcff44 into DigitalRuby:master Sep 15, 2021
vslee added a commit to vslee/ExchangeSharp that referenced this pull request Sep 22, 2021
- reverts change made in DigitalRuby#646
- while that change fixed the User Data Websocket for Binance, it is very binance specific
- OnUserDataWebSocketAsync() needs to be rewritten to be multi-exchange friendly
vslee added a commit that referenced this pull request Sep 22, 2021
- reverts change made in #646
- while that change fixed the User Data Websocket for Binance, it is very binance specific
- OnUserDataWebSocketAsync() needs to be rewritten to be multi-exchange friendly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants