-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
Binance.Net.Objects.Models.Futures.BinanceFuturesOrder
Incorrect json map
#1334
Comments
Binance.Net.Objects.Models.Futures.BinanceFuturesOrder
BUGBinance.Net.Objects.Models.Futures.BinanceFuturesOrder
Incorrect json map
Thank you. How about Coin-m Future? the same problem? |
@dante1989 I don't use Coin-M Future, I can't send real request right now. Look at the document, Coin-M Future has some fields that are different. Looks like |
Hm seems to mapped incorrectly yes. Not sure where that originates from. I'll have a look |
I've corrected the model(s), should be working correct now. Let me know if you have any issues |
For Example: How could I make new ItemOrderBinance (class BinanceFuturesOrder) by converting from Class BinanceUsdFuturesOrder or copying the same properties from BinanceUsdFuturesOrder. |
Describe the bug
FILLED
UsdFuturesApi.Trading.GetOrderAsync(symbol, orderId)
Here is raw response from /fapi/v1/order?symbol=BTCUSDT&orderid=123
You maped
cumQty
toQuantityFilled
. cumQty does not exist in response.You maped
executedQty
toLastFilledQuantity
that should beQuantityFilled
The text was updated successfully, but these errors were encountered: