Skip to content

fix: rename activationPrice to activatePrice in FuturesNewAlgoOrderParams - #648

Merged
tiagosiebler merged 2 commits into
sieblyio:masterfrom
DanZBo:master
Feb 23, 2026
Merged

fix: rename activationPrice to activatePrice in FuturesNewAlgoOrderParams#648
tiagosiebler merged 2 commits into
sieblyio:masterfrom
DanZBo:master

Conversation

@DanZBo

@DanZBo DanZBo commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix incorrect parameter name in FuturesNewAlgoOrderParams.

According to Binance documentation, the correct parameter name is activatePrice, not activationPrice.

Fixes #647

Additional Information

PR Checklist

As part of the PR, make sure you have:

  • No breaking changes / documented all breaking changes clearly.
  • Updated & checked that all tests pass.
  • Increased the version number in the package.json
  • Checked npm install runs without issue.
  • Included the package-lock.json, if it changed after npm install
  • Checked npm run build runs without issue.
  • Updated the endpoint map (optional, if you know how).
  • Run llms.txt generator (optional, if you know how).

@tiagosiebler tiagosiebler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is great, thank you

@tiagosiebler
tiagosiebler merged commit 1d7c50a into sieblyio:master Feb 23, 2026
4 of 5 checks passed
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.

Incorrect activationPrice param in FuturesNewAlgoOrderParams Interface

2 participants