clearing._messages: don't require `.symbol` in brokerd side error msgs

basic_buy_bot
Tyler Goodlet 2023-06-17 16:11:40 -04:00
parent 909f880211
commit 84613cd596
1 changed files with 5 additions and 2 deletions

View File

@ -242,7 +242,8 @@ class BrokerdStatus(Struct):
'closed', 'closed',
] ]
account: str # TODO: do we need this?
account: str | None = None,
name: str = 'status' name: str = 'status'
filled: float = 0.0 filled: float = 0.0
reason: str = '' reason: str = ''
@ -287,9 +288,11 @@ class BrokerdError(Struct):
''' '''
oid: str oid: str
symbol: str
reason: str reason: str
# TODO: drop this right?
symbol: str | None = None
# if no brokerd order request was actually submitted (eg. we errored # if no brokerd order request was actually submitted (eg. we errored
# at the ``pikerd`` layer) then there will be ``reqid`` allocated. # at the ``pikerd`` layer) then there will be ``reqid`` allocated.
reqid: int | str | None = None reqid: int | str | None = None