-
Notifications
You must be signed in to change notification settings - Fork 21
Added upgradable contraintLimits #798
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
Added upgradable contraintLimits #798
Conversation
so the idea is to have other genericscheme with different gov params which will white list the schemeConstraint contract ?
Please note that option 2 is already available and exist any how.. what is the rational/gain to add another way to do that ? |
Hi @orenyodfat, exactly the idea would be to use another generic scheme with super safe governance parameters. A proposal should not be able to pass faster then 14 days. The reasons why a separate GenericScheme might be helpful for me are:
|
this add another complexity to the system . another things to look on and monitor . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please emit an event on all updates functions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please base this pr vs master (or open a new one)
Uh oh!
There was an error while loading. Please reload this page.