-
Notifications
You must be signed in to change notification settings - Fork 279
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
fix: Correct the coin type in the derivation path for wallet examples #1089
fix: Correct the coin type in the derivation path for wallet examples #1089
Conversation
Thanks! Can you rebase on top of master? We recently merged a PR fixing the CI. |
a4933a9
to
bb6df4c
Compare
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK bb6df4c
Sadly this needs another rebase :/ |
fa5b07c
to
8d7e8dc
Compare
Done |
…t examples fix: Correct the coin type in the derivation path for the three wallet examples
8d7e8dc
to
21206fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 21206fe
Corrects the derivation path for the three wallet examples. For testnet, the coin type should be 1 instead of 0.
Replaces #1076