pikers
/
piker
3
0
Fork 0
Code Issues 2 Pull Requests 5 Packages Projects Releases Wiki Activity

Better commenting around order-mode error block

account_tests
Tyler Goodlet 2023-08-10 12:41:53 -04:00
parent 78178c2fb7
commit 077d9bf1d2
1 changed files with 15 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
piker/ui/order_mode.py
View File

@ -1088,9 +1088,16 @@ async def process_trade_msg(
mode.on_submit(oid) mode.on_submit(oid)
case Status(resp='error'): case Status(resp='error'):
# delete level line from view
# do all the things for a cancel:
# - drop order-msg dialog from client table
# - delete level line from view
mode.on_cancel(oid) mode.on_cancel(oid)
broker_msg = msg.brokerd_msg
# TODO: parse into broker-side msg, or should we
# expect it to just be **that** msg verbatim (since
# we'd presumably have only 1 `Error` msg-struct)
broker_msg: dict = msg.brokerd_msg
log.error( log.error(
f'Order {oid}->{resp} with:\n{pformat(broker_msg)}' f'Order {oid}->{resp} with:\n{pformat(broker_msg)}'
) )
@ -1111,8 +1118,12 @@ async def process_trade_msg(
case Status( case Status(
resp='triggered', resp='triggered',
# req=Order(exec_mode='live', action='alert') as req, # TODO # TODO: do the struct-msg version, blah blah..
req={'exec_mode': 'live', 'action': 'alert'} as req, # req=Order(exec_mode='live', action='alert') as req,
req={
'exec_mode': 'live',
'action': 'alert',
} as req,
): ):
# should only be one "fill" for an alert # should only be one "fill" for an alert
# add a triangle and remove the level line # add a triangle and remove the level line