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

Add custom hd path option #9367

Merged
merged 2 commits into from Nov 7, 2020

Conversation

PatrykLucka
Copy link
Contributor

When adding accounts from a Ledger wallet, there are 2 options for the HD path,

  • Ledger Live: m/44'/60'/0'/0/0
  • Legacy (MEW / MyCrypto): m/44'/60'/0'

This PR adds an option to specify custom HD path.
Screenshot 2020-09-07 at 11 56 33
Screenshot 2020-09-07 at 11 56 43
Fixes #6716
Fixes #7486
Fixes #7238

@PatrykLucka PatrykLucka requested a review from a team as a code owner September 7, 2020 11:09
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2020

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@PatrykLucka
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@brad-decker
Copy link
Contributor

I'm super stoked and excited about this PR, adding a long-requested feature from the community. 👏 👏 👏 Thanks @PatrykLucka!

Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

LGTM! Also tested locally, and it works.

@rekmarks rekmarks merged commit f30d261 into MetaMask:develop Nov 7, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Nov 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants