T. Todua
T. Todua
I dont think you understand my point. I understand that you included all fields, but I said that some of those fields seem mistake i.e. `address`, `amount/price/cost`.. which i asked...
I say, that on the exchanges where those fields are present, it's mistake and should be fixed separately, and that mistake shouldnt be incorporated into core. or at least, on...
@carlosmiei If you can, to review & merge this PR. as in currencies tests I need to have this exemplary structure in base. after that, the straightforward one: https://github.com/ccxt/ccxt/pull/14910
@carlosmiei would you have a time in upcoming days that you could review this PR? this will be also used in tests-pr, because the currency structure (which is created atm...
@carlosmiei
PR ready
@yuchiu503 unless you provide verbose log of failed request, it's hard to help. also, take a look in: https://docs.ccxt.com/#/README?id=api-key-permissions
.fetchPositionsForSymbol ('BTC/USDT:USDT') ``` [ { info: { symbol: "BTCUSDT", positionAmt: "0.003", entryPrice: "36632.8", breakEvenPrice: "36651.1164", markPrice: "36631.60000000", unRealizedProfit: "-0.00360000", liquidationPrice: "36059.49839357", leverage: "50", maxNotionalValue: "3000000", marginType: "isolated", isolatedMargin: "2.14901880", isAutoAddMargin:...
@carlosmiei PR ready eventually
@carlosmiei this PR is ready to be merged, non-related issues on travis. this is much comprehensive tests for fetchTrades, had revealed some buggy exchanges/implementations with the help of such tests....