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

support new smart wallet in Wallet UI #6127

Closed
turadg opened this issue Sep 4, 2022 · 3 comments
Closed

support new smart wallet in Wallet UI #6127

turadg opened this issue Sep 4, 2022 · 3 comments
Assignees
Labels
Dapp & UI Support enhancement New feature or request
Milestone

Comments

@turadg
Copy link
Member

turadg commented Sep 4, 2022

What is the Problem Being Solved?

#6084 changes the API for smart wallet. We need the Wallet UI to migrate to using it.

Description of the Design

Requirements

Non-requirements

  • supporting Smart Wallet and solo wallet at once
    • solo wallet will no longer have a UI; only smart wallet will

Ideas

Security Considerations

--

Test Plan

--

@turadg turadg added the enhancement New feature or request label Sep 4, 2022
@Tartuffo Tartuffo added this to the Mainnet 1 RC0 milestone Sep 5, 2022
@Tartuffo
Copy link
Contributor

Closed by #6134

@turadg
Copy link
Member Author

turadg commented Sep 15, 2022

@Tartuffo we left this one open until @rowgraus verifies it meets Product acceptance criteria. If you want to track that separately okay.

@Tartuffo
Copy link
Contributor

@Tartuffo we left this one open until @rowgraus verifies it meets Product acceptance criteria. If you want to track that separately okay.

Let's track testing separately. There should be a user story issue for this @rowgraus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dapp & UI Support enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants