Skip to content
This repository has been archived by the owner on Nov 27, 2024. It is now read-only.

Commit

Permalink
fix: update governance script to use more opinionated workflows (#502)
Browse files Browse the repository at this point in the history
**Motivation:**

This PR updates several functions on the governance script to target
more opinionated workflows rather than batch governance functions
together in a generalized way.

**Modifications:**

- Remove all initRole functions besides allowing users to initialize a
new role, grant this role to role holders (optional), and grant
permissions to this role (optional).
- Limit create strategies functions to a single function that allows
users to create a single new strategy and grant role permissions only
using that strategy
- Remove `revokePolicy` functions in favor of a single
`updateRoleDescriptionAndRoleHolders`. This allows users to update one
role description and grant and remove role holders of that created role.
- Update the `createAccountAndSetRolePermissions` function so the
computed account address is used for granting permissions and require
permissions to only be granted not removed.

**Result:**

This governance script will help users fall in the pit of success with
more opinionated ways to self-govern their instance without overloading
actions.
  • Loading branch information
AustinGreen authored Dec 10, 2023
1 parent bdae93d commit 2942d53
Show file tree
Hide file tree
Showing 2 changed files with 669 additions and 681 deletions.
Loading

0 comments on commit 2942d53

Please sign in to comment.