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

Secure seed during creating account #1290

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Secure seed during creating account #1290

wants to merge 9 commits into from

Conversation

azackmatoff
Copy link
Collaborator

@azackmatoff azackmatoff commented Jun 15, 2023 •
edited
Loading

Securing seed during creating account by a new UX.

Here's the flow:

Account created Secure your account
Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 03 36 Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 03 44
Instructions Keep your seed
Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 03 50 Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 03 57
Keep your seed Button Enabled Secure your account
Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 04 03 Simulator Screen Shot - iPhone 14 Plus - 2023-07-11 at 20 04 09

@azackmatoff azackmatoff self-assigned this Jun 15, 2023
@azackmatoff azackmatoff added A0-ux PR changes the UX B1-medium Elevates a release containing this PR to "medium priority" C0-breaksnothing PR does not introduce any breaking changes labels Jun 15, 2023
@clangenb
Copy link
Member

This looks already very good! but I will focus on the bug fixes for now in the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clangenb clangenb Awaiting requested review from clangenb

@Eldar2021 Eldar2021 Awaiting requested review from Eldar2021

@SourbaevaJanaraJ SourbaevaJanaraJ Awaiting requested review from SourbaevaJanaraJ

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
A0-ux PR changes the UX B1-medium Elevates a release containing this PR to "medium priority" C0-breaksnothing PR does not introduce any breaking changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UX for new feature: secure your wallet during sign-up
2 participants
@azackmatoff @clangenb