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

A new mediation resolution doesn't reset the trader's choices #5221

Open
huey735 opened this issue Feb 23, 2021 · 4 comments
Open

A new mediation resolution doesn't reset the trader's choices #5221

huey735 opened this issue Feb 23, 2021 · 4 comments
Labels

Comments

@huey735
Copy link
Member

huey735 commented Feb 23, 2021

Description

When, for some reason, a mediator needs to make a second resolution suggestion, the trader that accepted the first one doesn't get to reject the second suggestion.

Version

1.5.5

Steps to reproduce

This was done on Regtest

  1. The mediator suggests that BTC buyer get the trade amount.
  2. The BTC buyer accepts it.
  3. The mediator suggests a new resolution in which the BTC seller gets the trade amount.
  4. The BTC seller accepts this one.
  5. The BTC buyer doesn't get a chance to reject it as it's automatically accepted and the trade is closed.

Expected behaviour

The trader that accepted the first suggestion should also get a option to reject the second suggestion.

Actual behaviour

As soon as the trader that rejected the first suggestion accepts the second one, the payout transaction gets published.

Additional info

@ripcurlx ripcurlx added the a:bug label Feb 23, 2021
@BtcContributor
Copy link
Contributor

BtcContributor commented Feb 24, 2021

I am taking a look.

@huey735 not sure if I am understanding well the steps.
If between step 2 and 3 the BTC seller rejects the mediation, automatically an arbitration is started and in this case the outcome will be forced without any accept/reject by the parties.
So I am not sure how to recreate your scenario where a second mediation could be requested.

Could you please explain that with more details? Sorry about that. Thanks.

Edit: chat with huey clarified the scenario. I am taking a look.

@huey735
Copy link
Member Author

huey735 commented Feb 24, 2021

A rejection of mediation doesn't immediately trigger arbitration.

@BtcContributor
Copy link
Contributor

I coordinated with @jmacxx. He will take care of this issue. Thanks!

@ghost
Copy link

ghost commented Feb 28, 2021

Adding here so I don't forget. Requests from @huey735

  • Make the resolution suggestion separate from closing of the ticket so users can continue to chat with mediator.
  • It'd be great if after the DelayedPayoutTx is valid that the Traders had a way to move onto arbitration independently of the mediation ticket.

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

No branches or pull requests

3 participants