Skip to content
This repository was archived by the owner on Oct 7, 2024. It is now read-only.

Validate seed across all wordlists#77

Merged
Gudahtt merged 3 commits intoMetaMask:mainfrom
hladik-dan:master
Jan 4, 2022
Merged

Validate seed across all wordlists#77
Gudahtt merged 3 commits intoMetaMask:mainfrom
hladik-dan:master

Conversation

@hladik-dan
Copy link
Copy Markdown
Contributor

These changes allow to use seed from any BIP39 wordlist.

In order to spread adoption even further I think that we should allow to use words from all BIP39 wordlist languages.

@hladik-dan hladik-dan requested a review from a team as a code owner December 20, 2020 23:28
Rose2161
Rose2161 previously approved these changes May 16, 2021
@hladik-dan
Copy link
Copy Markdown
Contributor Author

@Rose2161 Can you review this PR again please?
I did the rebase so there are no conflicts now.

Rose2161
Rose2161 previously approved these changes Dec 30, 2021
@Gudahtt
Copy link
Copy Markdown
Member

Gudahtt commented Jan 4, 2022

Thanks for the contribution! This looks good once the lint error has been fixed.

@hladik-dan
Copy link
Copy Markdown
Contributor Author

@Gudahtt Fixed via yarn lint:fix. It should pass now.

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Gudahtt Gudahtt merged commit 8197ded into MetaMask:main Jan 4, 2022
@adonesky1 adonesky1 mentioned this pull request Apr 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants