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

Delete / Finalize Vaults #34

Open
0xbigz opened this issue Aug 22, 2023 · 0 comments
Open

Delete / Finalize Vaults #34

0xbigz opened this issue Aug 22, 2023 · 0 comments

Comments

@0xbigz
Copy link
Member

0xbigz commented Aug 22, 2023

sharing thoughts on topic raise by both @dannpl and @wphan

  • easy to support if vault.total_shares = 0 and vault.init_ts - now > MIN_VAULT_LIFETIME, then let anyone close out vault (and associated user account through cpi call)
  • force closure, when these invariants above aren't true, seems more difficult/complex
  • ensuring that vault.total_shares can get to 0 (rather than dust) should be examined during testing
@wphan wphan mentioned this issue Sep 11, 2023
2 tasks
@wphan wphan linked a pull request Sep 11, 2023 that will close this issue
2 tasks
@wphan wphan removed a link to a pull request Sep 11, 2023
2 tasks
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

No branches or pull requests

1 participant