Limit Order: Rename status to state and add live_ledger_data to MonitorLimitOrderRequest #130
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BRBussy
changed the title
Aligns documentation Go examples with the proto rename of status → state and LimitOrderStatus → LimitOrderState. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
BRBussy
approved these changes
Dec 2, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
LimitOrderStatusenum toLimitOrderStateto better reflect that it represents the life-cycle state of a limit orderstatusfield tostateon theLimitOrdermessagelive_ledger_datafield toMonitorLimitOrderRequestto enable fetching live ledger data when monitoring orders via streamingDetails
Enum and Field Rename
The
LimitOrderStatusenum has been renamed toLimitOrderStateto clarify its purpose as a life-cycle state indicator. The corresponding field onLimitOrderwas renamed fromstatustostate.MonitorLimitOrderRequest Enhancement
Added
live_ledger_databoolean field toMonitorLimitOrderRequest:true: fetches live ledger data for the orderfalse: returns only stored metadatalive_ledger_datais set totrue, live ledger data will be populated with each state update.Files Changed
proto/meshtrade/trading/limit_order/v1/limit_order.protoproto/meshtrade/trading/limit_order/v1/service.proto