ccxt 4.4.31 → 4.4.32

package/js/src/hitbtc.d.ts

@@ -158,7 +158,7 @@ export default class hitbtc extends Exchange {
      * @name hitbtc#fetchOrderBooks
      * @description fetches information on open orders with bid (buy) and ask (sell) prices, volumes and other data for multiple markets
      * @see https://api.hitbtc.com/#order-books
-     * @param {string[]|undefined} symbols list of unified market symbols, all symbols fetched if undefined, default is undefined
+     * @param {string[]} [symbols] list of unified market symbols, all symbols fetched if undefined, default is undefined
      * @param {int} [limit] max number of entries per orderbook to return, default is undefined
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} a dictionary of [order book structures]{@link https://docs.ccxt.com/#/?id=order-book-structure} indexed by market symbol
@@ -535,8 +535,8 @@ export default class hitbtc extends Exchange {
      * @name hitbtc#closePosition
      * @description closes open positions for a market
      * @see https://api.hitbtc.com/#close-all-futures-margin-positions
-     * @param symbol
-     * @param side
+     * @param {string} symbol unified ccxt market symbol
+     * @param {string} side 'buy' or 'sell'
      * @param {object} [params] extra parameters specific to the okx api endpoint
      * @param {string} [params.symbol] *required* unified market symbol
      * @param {string} [params.marginMode] 'cross' or 'isolated', default is 'cross'

package/js/src/hitbtc.js

@@ -1559,7 +1559,7 @@ export default class hitbtc extends Exchange {
      * @name hitbtc#fetchOrderBooks
      * @description fetches information on open orders with bid (buy) and ask (sell) prices, volumes and other data for multiple markets
      * @see https://api.hitbtc.com/#order-books
-     * @param {string[]|undefined} symbols list of unified market symbols, all symbols fetched if undefined, default is undefined
+     * @param {string[]} [symbols] list of unified market symbols, all symbols fetched if undefined, default is undefined
      * @param {int} [limit] max number of entries per orderbook to return, default is undefined
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} a dictionary of [order book structures]{@link https://docs.ccxt.com/#/?id=order-book-structure} indexed by market symbol
@@ -3580,8 +3580,8 @@ export default class hitbtc extends Exchange {
      * @name hitbtc#closePosition
      * @description closes open positions for a market
      * @see https://api.hitbtc.com/#close-all-futures-margin-positions
-     * @param symbol
-     * @param side
+     * @param {string} symbol unified ccxt market symbol
+     * @param {string} side 'buy' or 'sell'
      * @param {object} [params] extra parameters specific to the okx api endpoint
      * @param {string} [params.symbol] *required* unified market symbol
      * @param {string} [params.marginMode] 'cross' or 'isolated', default is 'cross'

package/js/src/htx.d.ts

@@ -264,7 +264,7 @@ export default class htx extends Exchange {
      * @see https://huobiapi.github.io/docs/spot/v1/en/#get-all-accounts-of-the-current-user
      * @param {string} type 'spot', 'swap' or 'future
      * @param {string} [marginMode] 'cross' or 'isolated'
-     * @param symbol
+     * @param {string} [symbol] unified ccxt market symbol
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} a dictionary of [account structures]{@link https://docs.ccxt.com/#/?id=account-structure} indexed by the account type
      */

package/js/src/htx.js

@@ -3215,7 +3215,7 @@ export default class htx extends Exchange {
      * @see https://huobiapi.github.io/docs/spot/v1/en/#get-all-accounts-of-the-current-user
      * @param {string} type 'spot', 'swap' or 'future
      * @param {string} [marginMode] 'cross' or 'isolated'
-     * @param symbol
+     * @param {string} [symbol] unified ccxt market symbol
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} a dictionary of [account structures]{@link https://docs.ccxt.com/#/?id=account-structure} indexed by the account type
      */

package/js/src/indodax.js

@@ -23,7 +23,7 @@ export default class indodax extends Exchange {
             'countries': ['ID'],
             // 10 requests per second for making trades => 1000ms / 10 = 100ms
             // 180 requests per minute (public endpoints) = 2 requests per second => cost = (1000ms / rateLimit) / 2 = 5
-            'rateLimit': 100,
+            'rateLimit': 50,
             'has': {
                 'CORS': undefined,
                 'spot': true,

package/js/src/kraken.d.ts

@@ -253,8 +253,8 @@ export default class kraken extends Exchange {
      * @name kraken#fetchOrdersByIds
      * @description fetch orders by the list of order id
      * @see https://docs.kraken.com/rest/#tag/Account-Data/operation/getClosedOrders
-     * @param {string[]|undefined} ids list of order id
-     * @param symbol
+     * @param {string[]} [ids] list of order id
+     * @param {string} [symbol] unified ccxt market symbol
      * @param {object} [params] extra parameters specific to the kraken api endpoint
      * @returns {object[]} a list of [order structure]{@link https://docs.ccxt.com/#/?id=order-structure}
      */
@@ -440,6 +440,8 @@ export default class kraken extends Exchange {
     parsePosition(position: Dict, market?: Market): import("./base/types.js").Position;
     parseAccountType(account: any): string;
     /**
+     * @method
+     * @name kraken#transferOut
      * @description transfer from spot wallet to futures wallet
      * @see https://docs.kraken.com/rest/#tag/User-Funding/operation/walletTransfer
      * @param {str} code Unified currency code

package/js/src/kraken.js

@@ -2147,8 +2147,8 @@ export default class kraken extends Exchange {
      * @name kraken#fetchOrdersByIds
      * @description fetch orders by the list of order id
      * @see https://docs.kraken.com/rest/#tag/Account-Data/operation/getClosedOrders
-     * @param {string[]|undefined} ids list of order id
-     * @param symbol
+     * @param {string[]} [ids] list of order id
+     * @param {string} [symbol] unified ccxt market symbol
      * @param {object} [params] extra parameters specific to the kraken api endpoint
      * @returns {object[]} a list of [order structure]{@link https://docs.ccxt.com/#/?id=order-structure}
      */
@@ -3071,6 +3071,8 @@ export default class kraken extends Exchange {
         return this.safeString(accountByType, account, account);
     }
     /**
+     * @method
+     * @name kraken#transferOut
      * @description transfer from spot wallet to futures wallet
      * @see https://docs.kraken.com/rest/#tag/User-Funding/operation/walletTransfer
      * @param {str} code Unified currency code

package/js/src/krakenfutures.d.ts

@@ -297,6 +297,8 @@ export default class krakenfutures extends Exchange {
     parseTransfer(transfer: Dict, currency?: Currency): TransferEntry;
     parseAccount(account: any): any;
     /**
+     * @method
+     * @name krakenfutures#transferOut
      * @description transfer from futures wallet to spot wallet
      * @param {str} code Unified currency code
      * @param {float} amount Size of the transfer

package/js/src/krakenfutures.js

@@ -2577,6 +2577,8 @@ export default class krakenfutures extends Exchange {
         }
     }
     /**
+     * @method
+     * @name krakenfutures#transferOut
      * @description transfer from futures wallet to spot wallet
      * @param {str} code Unified currency code
      * @param {float} amount Size of the transfer

package/js/src/kucoin.d.ts

@@ -42,9 +42,9 @@ export default class kucoin extends Exchange {
      */
     fetchMarkets(params?: {}): Promise<Market[]>;
     /**
-     * @param force
      * @method
      * @name kucoin#loadMigrationStatus
+     * @param {boolean} force load account state for non hf
      * @description loads the migration status for the account (hf or not)
      * @see https://www.kucoin.com/docs/rest/spot-trading/spot-hf-trade-pro-account/get-user-type
      */
@@ -742,8 +742,8 @@ export default class kucoin extends Exchange {
      * @name kucoin#setLeverage
      * @description set the level of leverage for a market
      * @see https://www.kucoin.com/docs/rest/margin-trading/margin-trading-v3-/modify-leverage-multiplier
-     * @param leverage
-     * @param {string} symbol unified market symbol
+     * @param {int } [leverage] New leverage multiplier. Must be greater than 1 and up to two decimal places, and cannot be less than the user's current debt leverage or greater than the system's maximum leverage
+     * @param {string} [symbol] unified market symbol
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} response from the exchange
      */

package/js/src/kucoin.js

@@ -574,6 +574,8 @@ export default class kucoin extends Exchange {
                     '400303': PermissionDenied,
                     '500000': ExchangeNotAvailable,
                     '260220': InvalidAddress,
+                    '600100': InsufficientFunds,
+                    '600101': InvalidOrder,
                     '900014': BadRequest, // {"code":"900014","msg":"Invalid chainId"}
                 },
                 'broad': {
@@ -1231,9 +1233,9 @@ export default class kucoin extends Exchange {
         return result;
     }
     /**
-     * @param force
      * @method
      * @name kucoin#loadMigrationStatus
+     * @param {boolean} force load account state for non hf
      * @description loads the migration status for the account (hf or not)
      * @see https://www.kucoin.com/docs/rest/spot-trading/spot-hf-trade-pro-account/get-user-type
      */
@@ -4950,8 +4952,8 @@ export default class kucoin extends Exchange {
      * @name kucoin#setLeverage
      * @description set the level of leverage for a market
      * @see https://www.kucoin.com/docs/rest/margin-trading/margin-trading-v3-/modify-leverage-multiplier
-     * @param leverage
-     * @param {string} symbol unified market symbol
+     * @param {int } [leverage] New leverage multiplier. Must be greater than 1 and up to two decimal places, and cannot be less than the user's current debt leverage or greater than the system's maximum leverage
+     * @param {string} [symbol] unified market symbol
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} response from the exchange
      */

package/js/src/kucoinfutures.d.ts

@@ -154,8 +154,8 @@ export default class kucoinfutures extends kucoin {
      * @description fetches historical positions
      * @see https://www.kucoin.com/docs/rest/futures-trading/positions/get-positions-history
      * @param {string[]} [symbols] list of unified market symbols
-     * @param since
-     * @param limit
+     * @param {int} [since] the earliest time in ms to fetch position history for
+     * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @param {int} [params.until] closing end time
      * @param {int} [params.pageId] page id
@@ -167,7 +167,7 @@ export default class kucoinfutures extends kucoin {
      * @method
      * @name kucoinfutures#createOrder
      * @description Create an order on the exchange
-     * @see https://docs.kucoin.com/futures/#place-an-order
+     * @see https://www.kucoin.com/docs/rest/futures-trading/orders/place-order
      * @see https://www.kucoin.com/docs/rest/futures-trading/orders/place-take-profit-and-stop-loss-order#http-request
      * @param {string} symbol Unified CCXT market symbol
      * @param {string} type 'limit' or 'market'
@@ -183,6 +183,7 @@ export default class kucoinfutures extends kucoin {
      * @param {bool} [params.reduceOnly] A mark to reduce the position size only. Set to false by default. Need to set the position size when reduceOnly is true.
      * @param {string} [params.timeInForce] GTC, GTT, IOC, or FOK, default is GTC, limit orders only
      * @param {string} [params.postOnly] Post only flag, invalid when timeInForce is IOC or FOK
+     * @param {float} [params.cost] the cost of the order in units of USDT
      * ----------------- Exchange Specific Parameters -----------------
      * @param {float} [params.leverage] Leverage size of the order
      * @param {string} [params.clientOid] client order id, defaults to uuid if not passed
@@ -355,6 +356,8 @@ export default class kucoinfutures extends kucoin {
      * @method
      * @name kucoinfutures#transfer
      * @description transfer currency internally between wallets on the same account
+     * @see https://www.kucoin.com/docs/rest/funding/transfer/transfer-to-main-or-trade-account
+     * @see https://www.kucoin.com/docs/rest/funding/transfer/transfer-to-futures-account
      * @param {string} code unified currency code
      * @param {float} amount amount to transfer
      * @param {string} fromAccount account to transfer from
@@ -365,6 +368,7 @@ export default class kucoinfutures extends kucoin {
     transfer(code: string, amount: number, fromAccount: string, toAccount: string, params?: {}): Promise<TransferEntry>;
     parseTransfer(transfer: Dict, currency?: Currency): TransferEntry;
     parseTransferStatus(status: Str): Str;
+    parseTransferType(transferType: Str): Str;
     /**
      * @method
      * @name kucoinfutures#fetchMyTrades
@@ -491,7 +495,7 @@ export default class kucoinfutures extends kucoin {
     setMarginMode(marginMode: string, symbol?: Str, params?: {}): Promise<MarginMode>;
     /**
      * @method
-     * @name kucoin#fetchLeverage
+     * @name kucoinfutures#fetchLeverage
      * @description fetch the set leverage for a market
      * @see https://www.kucoin.com/docs/rest/futures-trading/positions/get-cross-margin-leverage
      * @param {string} symbol unified market symbol

package/js/src/kucoinfutures.js

@@ -1226,8 +1226,8 @@ export default class kucoinfutures extends kucoin {
      * @description fetches historical positions
      * @see https://www.kucoin.com/docs/rest/futures-trading/positions/get-positions-history
      * @param {string[]} [symbols] list of unified market symbols
-     * @param since
-     * @param limit
+     * @param {int} [since] the earliest time in ms to fetch position history for
+     * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @param {int} [params.until] closing end time
      * @param {int} [params.pageId] page id
@@ -1429,7 +1429,7 @@ export default class kucoinfutures extends kucoin {
      * @method
      * @name kucoinfutures#createOrder
      * @description Create an order on the exchange
-     * @see https://docs.kucoin.com/futures/#place-an-order
+     * @see https://www.kucoin.com/docs/rest/futures-trading/orders/place-order
      * @see https://www.kucoin.com/docs/rest/futures-trading/orders/place-take-profit-and-stop-loss-order#http-request
      * @param {string} symbol Unified CCXT market symbol
      * @param {string} type 'limit' or 'market'
@@ -1445,6 +1445,7 @@ export default class kucoinfutures extends kucoin {
      * @param {bool} [params.reduceOnly] A mark to reduce the position size only. Set to false by default. Need to set the position size when reduceOnly is true.
      * @param {string} [params.timeInForce] GTC, GTT, IOC, or FOK, default is GTC, limit orders only
      * @param {string} [params.postOnly] Post only flag, invalid when timeInForce is IOC or FOK
+     * @param {float} [params.cost] the cost of the order in units of USDT
      * ----------------- Exchange Specific Parameters -----------------
      * @param {float} [params.leverage] Leverage size of the order
      * @param {string} [params.clientOid] client order id, defaults to uuid if not passed
@@ -1540,18 +1541,24 @@ export default class kucoinfutures extends kucoin {
         // required param, cannot be used twice
         const clientOrderId = this.safeString2(params, 'clientOid', 'clientOrderId', this.uuid());
         params = this.omit(params, ['clientOid', 'clientOrderId']);
-        if (amount < 1) {
-            throw new InvalidOrder(this.id + ' createOrder() minimum contract order amount is 1');
-        }
-        const preciseAmount = parseInt(this.amountToPrecision(symbol, amount));
         const request = {
             'clientOid': clientOrderId,
             'side': side,
             'symbol': market['id'],
             'type': type,
-            'size': preciseAmount,
             'leverage': 1,
         };
+        const cost = this.safeString(params, 'cost');
+        params = this.omit(params, 'cost');
+        if (cost !== undefined) {
+            request['valueQty'] = this.costToPrecision(symbol, cost);
+        }
+        else {
+            if (amount < 1) {
+                throw new InvalidOrder(this.id + ' createOrder() minimum contract order amount is 1');
+            }
+            request['size'] = parseInt(this.amountToPrecision(symbol, amount));
+        }
         const [triggerPrice, stopLossPrice, takeProfitPrice] = this.handleTriggerPrices(params);
         const stopLoss = this.safeDict(params, 'stopLoss');
         const takeProfit = this.safeDict(params, 'takeProfit');
@@ -2426,6 +2433,8 @@ export default class kucoinfutures extends kucoin {
      * @method
      * @name kucoinfutures#transfer
      * @description transfer currency internally between wallets on the same account
+     * @see https://www.kucoin.com/docs/rest/funding/transfer/transfer-to-main-or-trade-account
+     * @see https://www.kucoin.com/docs/rest/funding/transfer/transfer-to-futures-account
      * @param {string} code unified currency code
      * @param {float} amount amount to transfer
      * @param {string} fromAccount account to transfer from
@@ -2434,9 +2443,6 @@ export default class kucoinfutures extends kucoin {
      * @returns {object} a [transfer structure]{@link https://docs.ccxt.com/#/?id=transfer-structure}
      */
     async transfer(code, amount, fromAccount, toAccount, params = {}) {
-        if ((toAccount !== 'main' && toAccount !== 'funding') || (fromAccount !== 'futures' && fromAccount !== 'future' && fromAccount !== 'contract')) {
-            throw new BadRequest(this.id + ' transfer() only supports transfers from contract(future) account to main(funding) account');
-        }
         await this.loadMarkets();
         const currency = this.currency(code);
         const amountToPrecision = this.currencyToPrecision(code, amount);
@@ -2444,38 +2450,95 @@ export default class kucoinfutures extends kucoin {
             'currency': this.safeString(currency, 'id'),
             'amount': amountToPrecision,
         };
-        // transfer from usdm futures wallet to spot wallet
-        const response = await this.futuresPrivatePostTransferOut(this.extend(request, params));
-        //
-        //    {
-        //        "code": "200000",
-        //        "data": {
-        //            "applyId": "5bffb63303aa675e8bbe18f9" // Transfer-out request ID
-        //        }
-        //    }
-        //
-        const data = this.safeValue(response, 'data');
+        const toAccountString = this.parseTransferType(toAccount);
+        let response = undefined;
+        if (toAccountString === 'TRADE' || toAccountString === 'MAIN') {
+            request['recAccountType'] = toAccountString;
+            response = await this.futuresPrivatePostTransferOut(this.extend(request, params));
+            //
+            //     {
+            //         "code": "200000",
+            //         "data": {
+            //             "applyId": "6738754373ceee00011ec3f8",
+            //             "bizNo": "6738754373ceee00011ec3f7",
+            //             "payAccountType": "CONTRACT",
+            //             "payTag": "DEFAULT",
+            //             "remark": "",
+            //             "recAccountType": "MAIN",
+            //             "recTag": "DEFAULT",
+            //             "recRemark": "",
+            //             "recSystem": "KUCOIN",
+            //             "status": "PROCESSING",
+            //             "currency": "USDT",
+            //             "amount": "5",
+            //             "fee": "0",
+            //             "sn": 1519769124846692,
+            //             "reason": "",
+            //             "createdAt": 1731753283000,
+            //             "updatedAt": 1731753283000
+            //         }
+            //     }
+            //
+        }
+        else if (toAccount === 'future' || toAccount === 'swap' || toAccount === 'contract') {
+            request['payAccountType'] = this.parseTransferType(fromAccount);
+            response = await this.futuresPrivatePostTransferIn(this.extend(request, params));
+            //
+            //    {
+            //        "code": "200000",
+            //        "data": {
+            //            "applyId": "5bffb63303aa675e8bbe18f9" // Transfer-out request ID
+            //        }
+            //    }
+            //
+        }
+        else {
+            throw new BadRequest(this.id + ' transfer() only supports transfers between future/swap, spot and funding accounts');
+        }
+        const data = this.safeDict(response, 'data', {});
         return this.extend(this.parseTransfer(data, currency), {
             'amount': this.parseNumber(amountToPrecision),
-            'fromAccount': 'future',
-            'toAccount': 'spot',
+            'fromAccount': fromAccount,
+            'toAccount': toAccount,
         });
     }
     parseTransfer(transfer, currency = undefined) {
         //
-        // transfer
+        // transfer to spot or funding account
         //
         //     {
         //            "applyId": "5bffb63303aa675e8bbe18f9" // Transfer-out request ID
         //     }
         //
+        // transfer to future account
+        //
+        //     {
+        //         "applyId": "6738754373ceee00011ec3f8",
+        //         "bizNo": "6738754373ceee00011ec3f7",
+        //         "payAccountType": "CONTRACT",
+        //         "payTag": "DEFAULT",
+        //         "remark": "",
+        //         "recAccountType": "MAIN",
+        //         "recTag": "DEFAULT",
+        //         "recRemark": "",
+        //         "recSystem": "KUCOIN",
+        //         "status": "PROCESSING",
+        //         "currency": "USDT",
+        //         "amount": "5",
+        //         "fee": "0",
+        //         "sn": 1519769124846692,
+        //         "reason": "",
+        //         "createdAt": 1731753283000,
+        //         "updatedAt": 1731753283000
+        //     }
+        //
         const timestamp = this.safeInteger(transfer, 'updatedAt');
         return {
             'id': this.safeString(transfer, 'applyId'),
             'timestamp': timestamp,
             'datetime': this.iso8601(timestamp),
             'currency': this.safeCurrencyCode(undefined, currency),
-            'amount': undefined,
+            'amount': this.safeNumber(transfer, 'amount'),
             'fromAccount': undefined,
             'toAccount': undefined,
             'status': this.safeString(transfer, 'status'),
@@ -2488,6 +2551,13 @@ export default class kucoinfutures extends kucoin {
         };
         return this.safeString(statuses, status, status);
     }
+    parseTransferType(transferType) {
+        const transferTypes = {
+            'spot': 'TRADE',
+            'funding': 'MAIN',
+        };
+        return this.safeStringUpper(transferTypes, transferType, transferType);
+    }
     /**
      * @method
      * @name kucoinfutures#fetchMyTrades
@@ -2889,6 +2959,7 @@ export default class kucoinfutures extends kucoin {
         /**
          * @ignore
          * @method
+         * @name kucoinfutures#parseMarketLeverageTiers
          * @param {object} info Exchange market response for 1 market
          * @param {object} market CCXT market
          */
@@ -3130,7 +3201,7 @@ export default class kucoinfutures extends kucoin {
     }
     /**
      * @method
-     * @name kucoin#fetchLeverage
+     * @name kucoinfutures#fetchLeverage
      * @description fetch the set leverage for a market
      * @see https://www.kucoin.com/docs/rest/futures-trading/positions/get-cross-margin-leverage
      * @param {string} symbol unified market symbol

package/js/src/okx.d.ts

@@ -1042,8 +1042,8 @@ export default class okx extends Exchange {
      * @see https://www.okx.com/docs-v5/en/#trading-account-rest-api-get-bills-details-last-3-months
      * @param {string} [symbol] not used by okx fetchMarginAdjustmentHistory
      * @param {string} [type] "add" or "reduce"
-     * @param since
-     * @param limit
+     * @param {int} [since] the earliest time in ms to fetch margin adjustment history for
+     * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} params extra parameters specific to the exchange api endpoint
      * @param {boolean} [params.auto] true if fetching auto margin increases
      * @returns {object[]} a list of [margin structures]{@link https://docs.ccxt.com/#/?id=margin-loan-structure}

package/js/src/okx.js

@@ -8332,8 +8332,8 @@ export default class okx extends Exchange {
      * @see https://www.okx.com/docs-v5/en/#trading-account-rest-api-get-bills-details-last-3-months
      * @param {string} [symbol] not used by okx fetchMarginAdjustmentHistory
      * @param {string} [type] "add" or "reduce"
-     * @param since
-     * @param limit
+     * @param {int} [since] the earliest time in ms to fetch margin adjustment history for
+     * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} params extra parameters specific to the exchange api endpoint
      * @param {boolean} [params.auto] true if fetching auto margin increases
      * @returns {object[]} a list of [margin structures]{@link https://docs.ccxt.com/#/?id=margin-loan-structure}

package/js/src/phemex.js

@@ -491,6 +491,13 @@ export default class phemex extends Exchange {
                 'transfer': {
                     'fillResponseFromRequest': true,
                 },
+                'triggerPriceTypesMap': {
+                    'last': 'ByLastPrice',
+                    'mark': 'ByMarkPrice',
+                    'index': 'ByIndexPrice',
+                    'ask': 'ByAskPrice',
+                    'bid': 'ByBidPrice',
+                },
             },
         });
     }
@@ -2140,6 +2147,7 @@ export default class phemex extends Exchange {
             'PartiallyFilled': 'open',
             'Filled': 'closed',
             'Canceled': 'canceled',
+            'Suspended': 'canceled',
             '1': 'open',
             '2': 'canceled',
             '3': 'closed',
@@ -2646,17 +2654,11 @@ export default class phemex extends Exchange {
                     }
                     const stopLossTriggerPriceType = this.safeString2(stopLoss, 'triggerPriceType', 'slTrigger');
                     if (stopLossTriggerPriceType !== undefined) {
-                        if (market['settle'] === 'USDT') {
-                            if ((stopLossTriggerPriceType !== 'ByMarkPrice') && (stopLossTriggerPriceType !== 'ByLastPrice') && (stopLossTriggerPriceType !== 'ByIndexPrice') && (stopLossTriggerPriceType !== 'ByAskPrice') && (stopLossTriggerPriceType !== 'ByBidPrice') && (stopLossTriggerPriceType !== 'ByMarkPriceLimit') && (stopLossTriggerPriceType !== 'ByLastPriceLimit')) {
-                                throw new InvalidOrder(this.id + ' createOrder() take profit trigger price type must be one of "ByMarkPrice", "ByIndexPrice", "ByAskPrice", "ByBidPrice", "ByMarkPriceLimit", "ByLastPriceLimit" or "ByLastPrice"');
-                            }
-                        }
-                        else {
-                            if ((stopLossTriggerPriceType !== 'ByMarkPrice') && (stopLossTriggerPriceType !== 'ByLastPrice')) {
-                                throw new InvalidOrder(this.id + ' createOrder() take profit trigger price type must be one of "ByMarkPrice", or "ByLastPrice"');
-                            }
-                        }
-                        request['slTrigger'] = stopLossTriggerPriceType;
+                        request['slTrigger'] = this.safeString(this.options['triggerPriceTypesMap'], stopLossTriggerPriceType, stopLossTriggerPriceType);
+                    }
+                    const slLimitPrice = this.safeString(stopLoss, 'price');
+                    if (slLimitPrice !== undefined) {
+                        request['slPxRp'] = this.priceToPrecision(symbol, slLimitPrice);
                     }
                 }
                 if (takeProfitDefined) {
@@ -2670,19 +2672,13 @@ export default class phemex extends Exchange {
                     else {
                         request['takeProfitEp'] = this.toEp(takeProfitTriggerPrice, market);
                     }
-                    const takeProfitTriggerPriceType = this.safeString2(stopLoss, 'triggerPriceType', 'tpTrigger');
+                    const takeProfitTriggerPriceType = this.safeString2(takeProfit, 'triggerPriceType', 'tpTrigger');
                     if (takeProfitTriggerPriceType !== undefined) {
-                        if (market['settle'] === 'USDT') {
-                            if ((takeProfitTriggerPriceType !== 'ByMarkPrice') && (takeProfitTriggerPriceType !== 'ByLastPrice') && (takeProfitTriggerPriceType !== 'ByIndexPrice') && (takeProfitTriggerPriceType !== 'ByAskPrice') && (takeProfitTriggerPriceType !== 'ByBidPrice') && (takeProfitTriggerPriceType !== 'ByMarkPriceLimit') && (takeProfitTriggerPriceType !== 'ByLastPriceLimit')) {
-                                throw new InvalidOrder(this.id + ' createOrder() take profit trigger price type must be one of "ByMarkPrice", "ByIndexPrice", "ByAskPrice", "ByBidPrice", "ByMarkPriceLimit", "ByLastPriceLimit" or "ByLastPrice"');
-                            }
-                        }
-                        else {
-                            if ((takeProfitTriggerPriceType !== 'ByMarkPrice') && (takeProfitTriggerPriceType !== 'ByLastPrice')) {
-                                throw new InvalidOrder(this.id + ' createOrder() take profit trigger price type must be one of "ByMarkPrice", or "ByLastPrice"');
-                            }
-                        }
-                        request['tpTrigger'] = takeProfitTriggerPriceType;
+                        request['tpTrigger'] = this.safeString(this.options['triggerPriceTypesMap'], takeProfitTriggerPriceType, takeProfitTriggerPriceType);
+                    }
+                    const tpLimitPrice = this.safeString(takeProfit, 'price');
+                    if (tpLimitPrice !== undefined) {
+                        request['tpPxRp'] = this.priceToPrecision(symbol, tpLimitPrice);
                     }
                 }
             }

package/js/src/pro/binance.d.ts

@@ -167,7 +167,7 @@ export default class binance extends binanceRest {
      * @description watch the public liquidations of a trading pair
      * @see https://developers.binance.com/docs/derivatives/usds-margined-futures/websocket-market-streams/All-Market-Liquidation-Order-Streams
      * @see https://developers.binance.com/docs/derivatives/coin-margined-futures/websocket-market-streams/All-Market-Liquidation-Order-Streams
-     * @param {string[]} symbols
+     * @param {string[]} symbols list of unified market symbols
      * @param {int} [since] the earliest time in ms to fetch liquidations for
      * @param {int} [limit] the maximum number of liquidation structures to retrieve
      * @param {object} [params] exchange specific parameters for the bitmex api endpoint
@@ -195,7 +195,7 @@ export default class binance extends binanceRest {
      * @description watch the private liquidations of a trading pair
      * @see https://developers.binance.com/docs/derivatives/usds-margined-futures/user-data-streams/Event-Order-Update
      * @see https://developers.binance.com/docs/derivatives/coin-margined-futures/user-data-streams/Event-Order-Update
-     * @param symbols
+     * @param {string[]} symbols list of unified market symbols
      * @param {int} [since] the earliest time in ms to fetch liquidations for
      * @param {int} [limit] the maximum number of liquidation structures to retrieve
      * @param {object} [params] exchange specific parameters for the bitmex api endpoint
@@ -636,7 +636,7 @@ export default class binance extends binanceRest {
      * @see https://developers.binance.com/docs/binance-spot-api-docs/web-socket-api#cancel-order-trade
      * @see https://developers.binance.com/docs/derivatives/usds-margined-futures/trade/websocket-api/Cancel-Order
      * @param {string} id order id
-     * @param {string} symbol unified market symbol, default is undefined
+     * @param {string} [symbol] unified market symbol, default is undefined
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @param {string|undefined} [params.cancelRestrictions] Supported values: ONLY_NEW - Cancel will succeed if the order status is NEW. ONLY_PARTIALLY_FILLED - Cancel will succeed if order status is PARTIALLY_FILLED.
      * @returns {object} an list of [order structures]{@link https://docs.ccxt.com/#/?id=order-structure}
@@ -647,7 +647,7 @@ export default class binance extends binanceRest {
      * @name binance#cancelAllOrdersWs
      * @description cancel all open orders in a market
      * @see https://developers.binance.com/docs/binance-spot-api-docs/web-socket-api#cancel-open-orders-trade
-     * @param {string} symbol unified market symbol of the market to cancel orders in
+     * @param {string} [symbol] unified market symbol of the market to cancel orders in
      * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object[]} a list of [order structures]{@link https://docs.ccxt.com/#/?id=order-structure}
      */
@@ -659,8 +659,8 @@ export default class binance extends binanceRest {
      * @see https://developers.binance.com/docs/binance-spot-api-docs/web-socket-api#query-order-user_data
      * @see https://developers.binance.com/docs/derivatives/usds-margined-futures/trade/websocket-api/Query-Order
      * @param {string} id order id
-     * @param {string} symbol unified symbol of the market the order was made in
-     * @param {object} params extra parameters specific to the exchange API endpoint
+     * @param {string} [symbol] unified symbol of the market the order was made in
+     * @param {object} [params] extra parameters specific to the exchange API endpoint
      * @returns {object} An [order structure]{@link https://docs.ccxt.com/#/?id=order-structure}
      */
     fetchOrderWs(id: string, symbol?: Str, params?: {}): Promise<Order>;
@@ -727,8 +727,8 @@ export default class binance extends binanceRest {
      * @name binance#watchPositions
      * @description watch all open positions
      * @param {string[]|undefined} symbols list of unified market symbols
-     * @param since
-     * @param limit
+     * @param {number} [since] since timestamp
+     * @param {number} [limit] limit
      * @param {object} params extra parameters specific to the exchange API endpoint
      * @param {boolean} [params.portfolioMargin] set to true if you would like to watch positions in a portfolio margin account
      * @returns {object[]} a list of [position structure]{@link https://docs.ccxt.com/en/latest/manual.html#position-structure}