From ec6ec3d37c579d8c80ef004e4784500ec6d74c4d Mon Sep 17 00:00:00 2001 From: Victor Lee Date: Wed, 27 Apr 2022 16:48:50 -0700 Subject: [PATCH] fix AmountFilled in Binance user stream - different for trade vs order --- .../API/Exchanges/BinanceGroup/Models/UserDataStream.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ExchangeSharp/API/Exchanges/BinanceGroup/Models/UserDataStream.cs b/src/ExchangeSharp/API/Exchanges/BinanceGroup/Models/UserDataStream.cs index 26641434..ac8693c2 100644 --- a/src/ExchangeSharp/API/Exchanges/BinanceGroup/Models/UserDataStream.cs +++ b/src/ExchangeSharp/API/Exchanges/BinanceGroup/Models/UserDataStream.cs @@ -88,7 +88,7 @@ public ExchangeOrderResult ExchangeOrderResult Result = status, ResultCode = CurrentOrderStatus, Message = OrderRejectReason, // can use for multiple things in the future if needed - Amount = CumulativeFilledQuantity, + AmountFilled = TradeId != null ? LastExecutedQuantity : CumulativeFilledQuantity, Price = OrderPrice, AveragePrice = CumulativeQuoteAssetTransactedQuantity / CumulativeFilledQuantity, // Average price can be found by doing Z divided by z. OrderDate = CryptoUtility.UnixTimeStampToDateTimeMilliseconds(OrderCreationTime),