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

trade history shows ongoing trade as completed #4

Closed
jorian opened this issue Nov 12, 2019 · 4 comments
Closed

trade history shows ongoing trade as completed #4

jorian opened this issue Nov 12, 2019 · 4 comments
Assignees

Comments

@jorian
Copy link

jorian commented Nov 12, 2019

When a swap is started, the trade is immediately listed as 'Completed'. It should however show the intermediary steps of a trade.

image

while the actual trade shows:

image

@jorian
Copy link
Author

jorian commented Nov 12, 2019

Simplest solution in my opinion would be to show the status as Pending, and when a user clicks on it, he / she can track the status of the swap.

@particle4dev
Copy link
Owner

particle4dev commented Nov 12, 2019

@jorian I think it's a little confusing here
Right now, there is no concept of swap in the app
Only concept of orders (Please read atomiclabs#615)
So there is 2 type of orders: maker order and taker order
Your swap is taker order
and it's immediately listed as 'Completed' because it's filled
but its swap is still running
Does this make sense?

@jorian
Copy link
Author

jorian commented Nov 12, 2019

Yes it makes sense. Though I'm not sure if this is UX-friendly, since a swap can still fail. The user might think the order is completed, while the actual swap might have failed.

So, a concept of swap might actually be helpful, just for UX.

@jorian
Copy link
Author

jorian commented Nov 14, 2019

In the screenshot in the README of this repo, you can see how it was done in the first version of marketmaker:

image

Now I understand there are some more steps involved, so this might be an issue to be implemented on the longer term.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants