Skip to content

Add get order by ID (internal/external) endpoints #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dmitrykrasovskih merged 2 commits into from
Oct 28, 2025
Merged

Add get order by ID (internal/external) endpoints #61

dmitrykrasovskih merged 2 commits into from
Oct 28, 2025

Conversation

@alexex10
Copy link
Contributor

@alexex10 alexex10 commented Oct 28, 2025 •
edited
Loading

Changes

trading_client.account.get_order_by_id(order_id)
trading_client.account.get_order_by_external_id(external_order_id)

Resolves #47

@alexex10 alexex10 marked this pull request as ready for review October 28, 2025 08:19
ex10ded
ex10ded previously approved these changes Oct 28, 2025
View reviewed changes
@dmitrykrasovskih dmitrykrasovskih merged commit cc45097 into starknet Oct 28, 2025
7 checks passed
@dmitrykrasovskih dmitrykrasovskih deleted the add-get-order-by-id-endpoints branch October 28, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmitrykrasovskih dmitrykrasovskih dmitrykrasovskih approved these changes

@ex10ded ex10ded ex10ded left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add direct order retrieval endpoints

4 participants

@alexex10 @dmitrykrasovskih @ex10ded