Skip to content
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

Closed
offerm opened this issue Nov 18, 2018 · 4 comments
Closed

placeorder enhancements #690

offerm opened this issue Nov 18, 2018 · 4 comments
Assignees
Milestone

Comments

@offerm
Copy link
Contributor

offerm commented Nov 18, 2018

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).

@offerm offerm added this to the 1.0.0-alpha.5 milestone Nov 18, 2018
@sangaman
Copy link
Collaborator

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.

@kilrau kilrau modified the milestones: 1.0.0-alpha.5, 1.0.0-alpha.6 Nov 19, 2018
@offerm
Copy link
Contributor Author

offerm commented Nov 20, 2018

This wha I see today:

admins-MacBook-Pro-2:xud admin$ xucli buy 0.0025 LTC/BTC 1.105
swapped 0.001 LTC with peer order 62e45ed0-eccb-11e8-a9ee-5d8556a34cdc
remaining 0.0015 LTC entered the order book as 92a14980-eccb-11e8-bb8f-69b162a0c6a5
admins-MacBook-Pro-2:xud admin$ 

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 :

admins-MacBook-Pro-2:xud admin$ xucli buy 0.0005 LTC/BTC 1.105
swapped 0.0005 LTC with peer order 94dfeee0-eccb-11e8-a9ee-5d8556a34cdc
admins-MacBook-Pro-2:xud admin$ 

I even don't see my order ID.

@sangaman
Copy link
Collaborator

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.

@kilrau kilrau changed the title placeorder enhancments placeorder enhancements Dec 3, 2018
@kilrau
Copy link
Contributor

kilrau commented Dec 6, 2018

Moved this to #609 (comment), closing here.

@kilrau kilrau closed this as completed Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants