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

Allow to check with a 25th word #3

Open
chiva opened this issue Sep 15, 2020 · 2 comments
Open

Allow to check with a 25th word #3

chiva opened this issue Sep 15, 2020 · 2 comments

Comments

@chiva
Copy link

chiva commented Sep 15, 2020

Currently, the recovery check fails when using it with a passphrase.
Would be nice to include the functionality to correctly check the recovery mnemonic + passphrase (25th word)

@My1
Copy link

My1 commented Sep 29, 2020

problem would obviously be how to do it best UX wise, as a big part of the passphrase is that it may not be visible whether a passphrase is used.

@chiva
Copy link
Author

chiva commented Sep 29, 2020

Maybe ask after 24th word -> Use 25th word/passphrase? -> And then allow to add one if confirmed

Or add an item in the option menu of the Recovery Check app -> Advanced mode / Passphrase
This way, the default behaviour of 24 words would remain the same for standard users, so that it won't confuse them with the 25th/passphrase question, and only ask for it if it has been enabled in the options menu.

aido pushed a commit to aido/app-seed-tool that referenced this issue Apr 5, 2024
src: onboarding_seed_sskr.c: Use stable cx_crc32 instead of cx_crc32_hw
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants