Skip to content

fix: add Gtd variant to TimeInForce enum#13

Open
hotea wants to merge 1 commit into
mainfrom
fix/time-in-force-gtd
Open

fix: add Gtd variant to TimeInForce enum#13
hotea wants to merge 1 commit into
mainfrom
fix/time-in-force-gtd

Conversation

@hotea

@hotea hotea commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add Gtd variant (serialized as "GTD") to TimeInForce enum in src/model/enums.rs
  • Add serialize/deserialize tests for the new variant

Root Cause

TimeInForce only had Day, Gtc, Opg. Gtd (Good-Till-Date) was documented but missing from the enum, making it impossible to construct GTD orders with the typed enum.

Test plan

  • cargo test passes with new GTD serialize/deserialize assertions
  • Place a GTD order using TimeInForce::Gtd and verify it serializes to "GTD"

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