Skip to content

Add : Reduce only into arguments for limit_buy/limit_sell#290

Open
DevWonder01 wants to merge 3 commits into
ccxt:masterfrom
DevWonder01:algoOrder-fix
Open

Add : Reduce only into arguments for limit_buy/limit_sell#290
DevWonder01 wants to merge 3 commits into
ccxt:masterfrom
DevWonder01:algoOrder-fix

Conversation

@DevWonder01

Copy link
Copy Markdown

No description provided.

@DevWonder01 DevWonder01 changed the title Algo order fix Add : Reduce only into arguments for limit_buy/limit_sell Apr 20, 2026
@carlosmiei carlosmiei requested a review from Copilot June 2, 2026 14:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the FuturesAccount limit order helpers to optionally set Binance Futures’ reduceOnly flag, enabling callers to place reduce-only LIMIT BUY/SELL orders via the convenience methods.

Changes:

  • Add a reduce_only: Option<bool> argument to FuturesAccount::limit_buy and FuturesAccount::limit_sell.
  • Wire the new argument into the OrderRequest so reduceOnly is emitted when provided.
  • Minor whitespace cleanup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/futures/account.rs
Comment thread src/futures/account.rs
Comment on lines 278 to 282
pub fn limit_sell(
&self, symbol: impl Into<String>, qty: impl Into<f64>, price: f64,
time_in_force: TimeInForce,
reduce_only : Option<bool>
) -> Result<Transaction> {
Comment thread src/futures/account.rs
Comment on lines 247 to 251
pub fn limit_buy(
&self, symbol: impl Into<String>, qty: impl Into<f64>, price: f64,
time_in_force: TimeInForce,
reduce_only : Option<bool>,
) -> Result<Transaction> {
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants