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

Validate seed across all wordlists #77

Merged
merged 3 commits into from
Jan 4, 2022
Merged

Validate seed across all wordlists #77

merged 3 commits into from
Jan 4, 2022

Conversation

hladik-dan
Copy link
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
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
Member

Gudahtt commented Jan 4, 2022

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

@hladik-dan
Copy link
Contributor Author

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

Copy link
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