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

Revokable contract with Revoker Role #2037

Closed
wants to merge 3 commits into from
Closed

Revokable contract with Revoker Role #2037

wants to merge 3 commits into from

Conversation

litvinjuan
Copy link

This pull request corresponds to issue #2036
This issue has yet to be discussed, but as I had already written the code before opening it, I chose to create the PR for reference while discussing the issue.

Please, do read #2036 and join the discussion.

Fixes #

Changes for this PR are in the lifecycle and the access folders

  • Added RevokerRole contract in access
    This is a new role that corresponds to the Revokeble contract described below.

  • Added Revokable contract in lifecycle
    This contract allows any account with the Revoker role to call a revoke method, which permanently revokes the current contract. This adds a whenNotRevoked modifier that reverts if the contract is revoked. Revoking cannot be undone, similarly to Pausing, but permanent. (View Revokable contract with RevokerRole #2036)

@stale
Copy link

stale bot commented Jan 19, 2020

Hi all!
This Pull Request has not had any recent activity, is it still relevant? If so, what is blocking it? Is there anything we can do to help move it forward?
Thanks!

@stale stale bot added the stale label Jan 19, 2020
@stale
Copy link

stale bot commented Feb 3, 2020

Hi folks!
This Pull Request is being closed as there was no response to the previous prompt. However, please leave a comment whenever you're ready to resume, so it can be reopened.
Thanks again!

@stale stale bot closed this Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant