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

[feat]document ArbitrablePermissionList #82 #107

Merged
merged 2 commits into from
Aug 18, 2018

Conversation

yushih
Copy link
Contributor

@yushih yushih commented Aug 17, 2018

No description provided.

Copy link
Member

@clesaege clesaege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed one typo.


Also, if not append-only, the list allows anyone to preemptively clear an item not registered before by calling `requestClearing`. The process is also familiar: if a defined time period passes without dispute (by `challengeClearing`), the item could be cleared by calling `executeRequst`. Otherwise the arbitrator rules.

There is a flag `rechallengePossible`, when true, that could change the submission process slightly. When an item is submitted (by `requestRegistration`) and then challenged (by `challengeRegistration`), and the arbitrator rules in favor of the submitter, the item is not directly registered as described above. Instead the effect of the ruling is as if there has been no challenge at all, meaning the submitter, in order to get the item registered, still has to wait through the challenge period without any challenge occuring and then call `executeRequest`. And of course, if a challenge does occur and succeeds (the arbitrator rules that the item be cleared), the item is cleared. In other words, if the `rechallengePossible` flag is set, the only way the submitter registers an item is to wait through thex period unchallenged. The flag only applies in this situation and does not have effect anywhere else.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"thex" -> "the"

@clesaege clesaege merged commit 68701d8 into kleros:master Aug 18, 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

Successfully merging this pull request may close these issues.

2 participants