Open douggie opened 1 year ago
any comments? thoughts?
@makarid any thoughts?
think i will add 2 flagsPORTFOLIO_MARGIN
and INVERSE
so we can route new portfolio margin futures or spot orders to the POST /papi/v1/um/order
endpoints, as well as route the the coinmargin endpoints (i.e. inverse) https://binance-docs.github.io/apidocs/delivery/en/#new-order-trade and the current default usd margin endpoints, (i.e. linear) https://binance-docs.github.io/apidocs/futures/en/#new-order-trade
As Binance has separate base URL's (i.e. https://papi.binance.com) i controlled it by setting exchange specific parameters.
arggh tests failing on kraken, will try to take a look and fix them up.
Hi, binance's new portfolio margin 1.5 (offsetting spot and perp positions) requires new end points ((i.e https://papi.binance.com) to place, cancel orders etc etc (https://binance-docs.github.io/apidocs/pm/en/#change-log).
Unless there are any objections I am proposing I add the following methods to
org.knowm.xchange.binance.service.BinanceTradeServiceRaw
and then addUSE_PORTFOLIO_MARGIN
order flag toorg.knowm.xchange.binance.dto.trade.BinanceOrderFlags
to control the switching to the portfolio margin endpoints (i.e https://papi.binance.com) by using theUSE_PORTFOLIO_MARGIN
order flag inorg.knowm.xchange.binance.service.BinanceTradeService
org.knowm.xchange.binance.service.BinanceTradeServiceRaw
Unless that are any comments or objections I will assume that everyone is ok with me continuing with this approach.