We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation .
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
Windows 11
JavaScript
4.3.1
The data returned from ccxt does not provide a unified property for the closing time (i.e. in case of a limit order, the opening time and the moment in which it would be filled will differ).
E.g. see this slightly edited order structure from Coinbase. A kraken order also had an undefined "lastTradeTimestamp" (if that is meant to be the closing time)
{ "info" : { ... }, "id" : " ... " , "clientOrderId" : " ... " , "timestamp" : 1705100494781 , "datetime" : " 2024-01-12T23:01:34.781057Z " , "lastTradeTimestamp" : " undefined " , "symbol" : " ETH/EUR " , "type" : " limit " , "timeInForce" : " GTC " , "postOnly" : false , "side" : " buy " , "price" : 2286.78 , "stopPrice" : " undefined " , "triggerPrice" : " undefined " , "amount" : 0.00867595 , "filled" : 0.00867595 , "remaining" : 0 , "cost" : 19.839988941 , "average" : 2286.78 , "status" : " closed " , "fee" : { "cost" : " 0.119039933646 " , "currency" : " EUR " }, "trades" : [], "fees" : [ { "cost" : 0.119039933646 , "currency" : " EUR " } ], "lastUpdateTimestamp" : " undefined " , "reduceOnly" : " undefined " , "takeProfitPrice" : " undefined " , "stopLossPrice" : " undefined " }
Timestamp provides the opening time. The closing time is not present in the returned structure and needs to be found in the raw "info" object on a per exchange basis.
The text was updated successfully, but these errors were encountered:
Hello @Nomad-Soul , thanks for your suggestion we will take it into consideration and analyze if we can add it to the Order structure.
Sorry, something went wrong.
No worries. If you can agree on the name and location of the property I can try to add its initialisation for Coinbase and Kraken
carlosmiei
No branches or pull requests
