-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
placeorder enhancements #690
Comments
What specifically do you have in mind besides #609 (failed swap events)? That would then cover all matches, swaps, and failed swaps, plus any remaining quantity that gets added to the order book. |
This wha I see today:
I don't see the matches (it was matched against 2 orders) and I don't see the failure. I also don't see that the peer orders is removed from the book or that the quantity reduced. If I do that :
I even don't see my order ID. |
The failed swap events are being handled in #609 as I said. Is there a successful match/swap that you expected to see that wasn't there? Or was the 2nd match the failed one? We can return the orderId even when fully matched, that's not a problem. If we just want to make this issue about returning that orderId, then after this and #609 is solved it sounds like you'll have everything you're looking for. |
Moved this to #609 (comment), closing here. |
place order should be enhanced to show all events from order entry until order is entered to the book as ownorder.
we should always present the orderId (even if no own order created).
The text was updated successfully, but these errors were encountered: