-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Allow invalid bip39 mnemonic, add cosigner #60
Allow invalid bip39 mnemonic, add cosigner #60
Conversation
…entials when cloning the repo.
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.
Why default to testnet?
linking test report of my own I did with Guardian and GC: https://gist.github.com/gorazdko/0c3f04f50962235e0439be6398e83393 ready for review |
tACK |
@gorazdko, ready for me to merge? |
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.
Will merge with tACK.
@ChristopherA please do merge. There are also 4 PRs in bc-keytool-cli waiting to be merged. |
Abstract
Supercedes #58
The name chosen for the procedure is
BIP39 sentence completion
. The name and word usage is not set in stone and will be updated according to theresolution of BlockchainCommons/Gordian-Developer-Community#27
The guide can be seen rendered here and uses a reference to Rudefox ShowMyWork lookup tables:
https://github.com/gorazdko/bc-lethekit/blob/dd3d47e7704e5856d3327c0e16a97063b40f3a57/seedtool/doc/bip39_sentence_completion.md
Added cosigner
Also improvements in documentation:
Next steps
so that user can follow them more easilyStatus
ready for review