Skip to content

Conversation

@yzh-pelle
Copy link
Contributor

No description provided.

{
$request = [];
if ($algoId) {
$request['algoId'] = $algoId;
Copy link
Collaborator

Choose a reason for hiding this comment

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

@yzh-pelle u sure it is not algoid lowercase?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I checked both algoId and clientAlgoId. Both works good in camel case.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@yzh-pelle but we should follow the docs
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

hum weird, I'm pretty sure the exchange is tripping here does not make sense to have both (unless they updated the docs in the meantime)

@carlosmiei carlosmiei merged commit 5539e0a into ccxt:master Dec 10, 2025
1 check failed
@yzh-pelle yzh-pelle deleted the algo-orders-update branch December 10, 2025 21:10
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