T. Todua

Results 545 comments of 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...

@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....