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

arbitrary well-known ERTP asset support in smart wallet #8156

Closed
8 tasks done
dckc opened this issue Aug 7, 2023 · 0 comments · Fixed by #8147
Closed
8 tasks done

arbitrary well-known ERTP asset support in smart wallet #8156

dckc opened this issue Aug 7, 2023 · 0 comments · Fixed by #8147
Assignees
Labels
enhancement New feature or request

Comments

@dckc
Copy link
Member

dckc commented Aug 7, 2023

What is the Problem Being Solved?

The smart wallet supports vbank assets only (i.e. cosmos x/bank accounts reflected as ERTP purses); this suffices for the PSM and Vaults application, but not arbitrary ERTP assets as requested in #7226.

Description of the Design

Tasks

Preview Give feedback

Security Considerations

Forgery risks around issuer/brand naming (#7226, #4819, #1449) are avoided by only supporting issuers from agoricNames.

Scaling Considerations

n/a?

Test / Upgrade Considerations

noted in the design list above

@dckc dckc added the enhancement New feature or request label Aug 7, 2023
@dckc dckc changed the title address smart wallet limitation to vbank assets handle arbitrary ERTP assets in smart wallet Aug 7, 2023
@dckc dckc changed the title handle arbitrary ERTP assets in smart wallet arbitrary ERTP asset support in smart wallet Aug 7, 2023
@dckc dckc changed the title arbitrary ERTP asset support in smart wallet arbitrary well-known ERTP asset support in smart wallet Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant