Skip to content

Conversation

@lethaale
Copy link

https://api.docs.extended.exchange/#get-trades
params market is not required => set as Optional

https://api.docs.extended.exchange/#get-fees
params market is not required => set as Optional

https://api.docs.extended.exchange/#get-current-leverage
params market is not required => set as Optional

https://api.docs.extended.exchange/#get-candles-history
params limit is required => removed Optional

Copilot AI review requested due to automatic review settings January 21, 2026 13:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates parameter types in several API client methods to align with the official Extended Exchange API documentation. The changes make optional parameters that were incorrectly marked as required, and mark one previously optional parameter as required according to the API specification.

Changes:

  • Updated get_candles_history to make the limit parameter required (removed Optional)
  • Updated get_trades, get_fees, and get_leverage to make the market_names parameter optional
  • Added a helpful comment in an example file suggesting testnet configuration option

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
x10/perpetual/trading_client/markets_information_module.py Made limit parameter required in get_candles_history method
x10/perpetual/trading_client/account_module.py Made market_names parameter optional in get_trades, get_fees, and get_leverage methods
examples/01_create_limit_order.py Added clarifying comment about testnet configuration option

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

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.

1 participant