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

build: gov wrappers #63

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

build: gov wrappers #63

wants to merge 4 commits into from

Conversation

Schlagonia
Copy link
Collaborator

RoleManagerGovernance

  • Holds the daddy/governance role of the RoleManager contract who controls V3 multi strategy vaults.
  • Chad will be the governance address of this contract
  • Is a pass through for the majority of functions besides ones that would allow for governance to one step rug vaults.
  • Makes both the daddy and brain roles immutable,
  • This will also be done while giving SAM the max_debt role and removing it from Chad along with other roles that could allow a txn to touch user funds

V2VaultGovernance

  • Will be the governance address on each V2 vault
  • Pass through for most funcitons with additional lock or immutability of some
  • Adds a timelock of 7 days before adding a new strategy. Permissionless to add once timelock ends
  • Adds the ability to permanently lock the ability to ever add a new strategy
  • Makes governance and management roles immutable
  • Should it not allow to change the rewards address
  • Should not allow migrations of strategies?
  • Should not allow updating a strategies debt ratio so that sms must be involved even after a timelock

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.

1 participant