From 84613cd5960ee4f48c10d6a70f0ec097e001726c Mon Sep 17 00:00:00 2001 From: Tyler Goodlet Date: Sat, 17 Jun 2023 16:11:40 -0400 Subject: [PATCH] clearing._messages: don't require `.symbol` in brokerd side error msgs --- piker/clearing/_messages.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/piker/clearing/_messages.py b/piker/clearing/_messages.py index 240c340b..21974971 100644 --- a/piker/clearing/_messages.py +++ b/piker/clearing/_messages.py @@ -242,7 +242,8 @@ class BrokerdStatus(Struct): 'closed', ] - account: str + # TODO: do we need this? + account: str | None = None, name: str = 'status' filled: float = 0.0 reason: str = '' @@ -287,9 +288,11 @@ class BrokerdError(Struct): ''' oid: str - symbol: str reason: str + # TODO: drop this right? + symbol: str | None = None + # if no brokerd order request was actually submitted (eg. we errored # at the ``pikerd`` layer) then there will be ``reqid`` allocated. reqid: int | str | None = None