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.

Sign up for GitHub

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

Jump to bottom

Simplify DelayedAdmin/PauseAdmin #249

Open
hieronx opened this issue Mar 8, 2024 · 0 comments
Open

Simplify DelayedAdmin/PauseAdmin #249

hieronx opened this issue Mar 8, 2024 · 0 comments
Assignees
@AStox
Labels
I6-refactoring Code needs refactoring. Q3-medium Can be done with good experience in the language, but little knowledge of the codebase.

Comments

@hieronx
Copy link
Contributor

hieronx commented Mar 8, 2024 •
edited
Loading

One idea would be to have a single Guardian contract, that has a single account which is the current DelayedAdmin multisig, and that uses the Gnosis Safe isOwner() method to allow individual owners on the multisig to pause

Draft: https://github.com/centrifuge/liquidity-pools/blob/guardian-wip/src/admins/DelayedAdmin.sol
@hieronx hieronx added I6-refactoring Code needs refactoring. Q3-medium Can be done with good experience in the language, but little knowledge of the codebase. labels Mar 8, 2024
@AStox AStox self-assigned this Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AStox AStox

Labels
I6-refactoring Code needs refactoring. Q3-medium Can be done with good experience in the language, but little knowledge of the codebase.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hieronx @AStox