-
Notifications
You must be signed in to change notification settings - Fork 222
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
Fire events when swap status changes. #645
Comments
I am sending all state changes to a swap, but you need to key on requestid/quoteid then each time a new tx comes in, you will see the event |
the undefined is probably from one of the logging events that wasnt via the command processing |
Filtering by |
here is a raw log of events: https://komodo-platform.slack.com/files/U02QSHX81/F9KQX8459/-.sh there are plenty of "update" method printouts. ignore the garbage at the end, that is due to removing the null char at the end, need to fix the echo app |
are you sure you are running the latest version with the "update" method in it? |
Resolved #643 (comment) |
Doing a trade that I have issues, if I monitor the socket for events that match the
tradeid
I just started I get the following messages in this exact order:Method:
"request"
Method:
"request"
Method:
"reserved"
Method:
"connect"
Method:
"connect"
Method:
"connected"
Method:
undefined
Method:
"tradestatus"
This is looking much better than before, however I'm not 100% sure what all the events are signifying. Especially the one with an undefined
method
property. Are you able to clarify on this?This looks like it allows us to reliably know when a trade is pending/complete. However, it doesn't provide much insight into the actual state/progress of the atomic swap.
With our (inefficient) polling solution we would get realtime updates on bobdeposit/alicepayment/bobpayment/paymentspent status.
It would be great if you could fire a new event every time there is new progress with the swap.
So each time the trade progresses, e.g new bobdeposit/alicepayment/bobpayment/paymentspent activity, we get a new event, and can update the UI on the progress of the trade.
Is this data that you have internally in mm? That you can just fire over to us when you get it?
The text was updated successfully, but these errors were encountered: