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

Implement AppealableArbitrator.sol #3

Closed
clesaege opened this issue Nov 6, 2017 · 8 comments
Closed

Implement AppealableArbitrator.sol #3

clesaege opened this issue Nov 6, 2017 · 8 comments
Assignees

Comments

@clesaege
Copy link
Member

clesaege commented Nov 6, 2017

This is a centralized arbitrator who can be appealed to another arbitrator.
The other arbitrator can be any kind of arbitrator.
This contract can inherit from CentralizedArbitrator.

@mahadevan-k
Copy link

sent you a pull request on this

@clesaege
Copy link
Member Author

clesaege commented Feb 7, 2018

@clesaege
Copy link
Member Author

clesaege commented Feb 7, 2018

Few comments about how to implement it (but if you have something else which would work that's fine too):
-The Appealable contract must designate another Arbitrator contract for appeals.
-You can get the cost of appeal by asking the other Arbitrator contract.
-After a ruling is made, you need to let some time for parties to appeal, otherwise if you call directly the arbitrable contract, the ruling will be enforced without possibility of appeal.

@mahadevan-k
Copy link

Got it, will look into this

@mahadevan-k
Copy link

For assigning the Arbitrator contract, I could add an assignDispute method and call it from appealSuccessful, but I'm wondering whether I should add it to AppealableArbitrator or Arbitrator.

Regarding the ruling, I'll move the rule method inside the else clause and add a appealTimeout method which will execute the ruling

@clesaege
Copy link
Member Author

AppealableArbitrator, after an appeal is made, need to transfer the dispute to the appeal arbitrator, so it is in AppealableArbitrator. So AppealableArbitrator must also be Arbitrable.

@0xferit
Copy link
Member

0xferit commented Aug 6, 2018

Looks like @bitonator gave up. I am starting to work on this.

@clesaege
Copy link
Member Author

clesaege commented Aug 6, 2018

You may want to look at #4 as they are quite similar.

@epiqueras epiqueras assigned epiqueras and 0xferit and unassigned epiqueras Sep 14, 2018
clesaege pushed a commit that referenced this issue Jul 6, 2020
Updating to the latest commit as of 29th June 2020
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

4 participants