Skip to content

feat(trade): re-enable all_executions across all SDKs - #559

Draft
sunli829 wants to merge 5 commits into
mainfrom
feat/enable-all-executions
Draft

feat(trade): re-enable all_executions across all SDKs#559
sunli829 wants to merge 5 commits into
mainfrom
feat/enable-all-executions

Conversation

@sunli829

Copy link
Copy Markdown
Collaborator

Summary

Re-enable the previously disabled all_executions API across all language SDKs now that the API is available.

Changes

  • Uncomment all_executions implementations in Rust, blocking Rust, Python (sync & async), Java, Node.js, C, and C++ SDKs
  • Restore Python type stub (openapi.pyi) for both TradeContext and AsyncTradeContext
  • Fix unused imports surfaced by clippy

Test plan

  • Verify all_executions returns data against live API

🤖 Generated with Claude Code

@sunli829
sunli829 marked this pull request as draft July 22, 2026 09:34
sunli829 added 4 commits July 30, 2026 12:10
The API is now available. Uncomment the previously disabled
all_executions implementations in Rust, blocking, Python (sync/async),
Java, Node.js, C, and C++ SDKs, and restore the Python type stub.
The today/history/all executions responses now return the buy/sell
direction of each fill. Expose it as `Execution.side` (`OrderSide`) in
every binding: Rust core, Python, Node.js, C, C++, and Java. Regenerate
the C header (cbindgen) and Node.js bindings.
@sunli829
sunli829 force-pushed the feat/enable-all-executions branch from ac34d74 to cbd0dac Compare July 30, 2026 04:20
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