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

Add deprecateDomain #983

Merged
merged 5 commits into from
Oct 19, 2021
Merged

Add deprecateDomain #983

merged 5 commits into from
Oct 19, 2021

Conversation

kronosapiens
Copy link
Member

@kronosapiens kronosapiens commented Aug 19, 2021

Closes #979

Allow for one-off domain deprecation (i.e. deprecating a domain does not deprecate the parents or children).

A deprecated domain cannot:

  • Create new child domains
  • Create new expenditures
  • Have funds transferred into it

I also took advantage of the opportunity to remove some lingering msg.sender calls which we no longer need.

contracts/colony/Colony.sol Outdated Show resolved Hide resolved
contracts/colony/ColonyAuthority.sol Outdated Show resolved Hide resolved
contracts/colony/ColonyFunding.sol Show resolved Hide resolved
contracts/colony/Colony.sol Outdated Show resolved Hide resolved
@kronosapiens kronosapiens force-pushed the feat/deprecate-domains branch from 1af9627 to a374825 Compare October 17, 2021 11:57
@kronosapiens kronosapiens force-pushed the feat/deprecate-domains branch from 7c36c9e to d0ee8b6 Compare October 17, 2021 12:05
@kronosapiens kronosapiens force-pushed the feat/deprecate-domains branch from d0ee8b6 to 6977d09 Compare October 17, 2021 12:14
@area area force-pushed the feat/deprecate-domains branch from 02841c2 to 55de7ca Compare October 18, 2021 10:10
@area area merged commit b907359 into develop Oct 19, 2021
@area area deleted the feat/deprecate-domains branch October 19, 2021 10:30
@kronosapiens kronosapiens mentioned this pull request Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow colonies to "deprecate" domains
2 participants