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

refine bankManager / VBANK to provide purses on demand #5412

Closed
dckc opened this issue May 20, 2022 · 1 comment
Closed

refine bankManager / VBANK to provide purses on demand #5412

dckc opened this issue May 20, 2022 · 1 comment
Labels
duplicate enhancement New feature or request needs-design performance Performance related issues wallet

Comments

@dckc
Copy link
Member

dckc commented May 20, 2022

What is the Problem Being Solved?

#5397 adds a permissionless (though expensive) way to call E(bankManager).addAsset(). Currently, that results in a new purse in every user's wallet.

The purses should only be created on demand.

Description of the Design

TBD. @michaelfig and I discussed a few options... nothing detailed yet.

Security Considerations

?

Test Plan

TBD
@dckc dckc added enhancement New feature or request wallet performance Performance related issues needs-design labels May 20, 2022
@dckc dckc mentioned this issue May 20, 2022
Merged
2 tasks
@warner warner mentioned this issue Aug 3, 2022
Closed
@dckc dckc mentioned this issue Jan 23, 2023
Open
@dckc
Copy link
Member Author

dckc commented Jan 23, 2023

overtaken by #6652

@dckc dckc closed this as completed Jan 23, 2023
@dckc dckc added the duplicate label Jan 23, 2023
@dckc dckc mentioned this issue Feb 8, 2023
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate enhancement New feature or request needs-design performance Performance related issues wallet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dckc @Tartuffo