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

VaultManager migrate to non-deprecated Notifier #10095

Open
Chris-Hibbert opened this issue Sep 16, 2024 · 0 comments
Open

VaultManager migrate to non-deprecated Notifier #10095

Chris-Hibbert opened this issue Sep 16, 2024 · 0 comments
Assignees
Labels
contract-upgrade enhancement New feature or request hygiene Tidying up around the house notifier Vaults VaultFactor (née Treasury)

Comments

@Chris-Hibbert
Copy link
Contributor

What is the Problem Being Solved?

VaultManager uses a StoredPublisherKit to provide access to changes in governed parameters. StoredPublisherKit is deprecated.

Description of the Design

Add a new method that uses a newer notifier. Deprecate and eventually remove the old.

Security Considerations

None

Scaling Considerations

Not particularly a scaling issue.

Test Plan

ordinary regression tests.

Upgrade Considerations

Take a couple of releases to remove the old. Ensure the UIs keep up.

@Chris-Hibbert Chris-Hibbert added enhancement New feature or request notifier hygiene Tidying up around the house contract-upgrade Vaults VaultFactor (née Treasury) labels Sep 16, 2024
@Chris-Hibbert Chris-Hibbert self-assigned this Sep 16, 2024
@Chris-Hibbert Chris-Hibbert changed the title VaultManager migtrate to non-deprecated Notifier VaultManager migrate to non-deprecated Notifier Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contract-upgrade enhancement New feature or request hygiene Tidying up around the house notifier Vaults VaultFactor (née Treasury)
Projects
None yet
Development

No branches or pull requests

1 participant