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

make Smart Wallet contract durable #5894

Closed
Tracked by #6644
turadg opened this issue Aug 4, 2022 · 0 comments · Fixed by #6871
Closed
Tracked by #6644

make Smart Wallet contract durable #5894

turadg opened this issue Aug 4, 2022 · 0 comments · Fixed by #6871
Assignees
Labels
enhancement New feature or request vaults_triage DO NOT USE wallet

Comments

@turadg
Copy link
Member

turadg commented Aug 4, 2022

All the wallet backends need to be implemented by a single wallet manager contract, since it is inefficient to have many small vats when a large one would suffice. Multitenant backend wallet states, which are managed by a single Zoe contract, are presented as sealed individual backends, each of the schema in #4483, and broadcasted via the Cosmos event stream.

To facilitate migration to virtual/durable objects and collection, each individual wallet backend must be supplied with a new schema manager rather than instantiating its own collections.

While refactoring, consider moving address out of smartWallet scope. E.g. pass in a wallet logger and storageNode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request vaults_triage DO NOT USE wallet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants