Allow `None` for `BrokerdError.reqid`
Found this caused breakage on `kraken` orders which triggered the "insufficient funds" error response. Makes sense since they won't generate an order id if the order can't ever be submitted.no_orderid_in_error
parent
84399e8131
commit
34e6db6d98
|
@ -242,7 +242,7 @@ class BrokerdError(BaseModel):
|
||||||
|
|
||||||
# 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: Union[int, str] = ''
|
reqid: Optional[Union[int, str]] = None
|
||||||
|
|
||||||
symbol: str
|
symbol: str
|
||||||
reason: str
|
reason: str
|
||||||
|
|
Loading…
Reference in New Issue