-
Notifications
You must be signed in to change notification settings - Fork 93
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
Derivation path #495
Comments
@aguycalled we should add an issue over at NavPay as well, and Next? We only have 3 wallets to update right? NEXT uses core? So do we have anything to change on that end? |
NEXT uses core, NavPay should see an issue opened too |
ACK |
I think NavPay is easier as it's just changing the default on creation and the default on import Then adding a note on import page. |
This statement was WRONG 👍 |
i do not see anything on this here: https://github.com/bitcoin/bips/blob/master/bip-0130.mediawiki |
and i found this: navcoin-core/src/wallet/wallet.cpp Line 119 in 498e75f
so what is it you finally wanna? |
you are right, updated issue description |
it'll be 44'/130'/0'/0'/0' EDIT: |
@alexeyneu What do you mean by "letter stuff" |
@alexeyneu are you working on this? If so I can assign this issue to you. |
may be i'll do it , may be not |
Currently navcoin core shares with bitcoin the derivation path used for the generation of new addresses (m/0'/0'), this should be changed to m/44'/130' while preserving backwards compatibility with old wallets.
The text was updated successfully, but these errors were encountered: