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

Fix derivation from xpubs #29

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Fix derivation from xpubs #29

wants to merge 6 commits into from

Conversation

alecov
Copy link

@alecov alecov commented Oct 8, 2021

Hi @meherett. This is the patch we've been discussing. Please check to see if it fits.

This small patch fixes a check in order to allow arbitrary xpubs to be
derived from without the need for root xpub/xprvs.
- New entries in cryptocurrencies.py;
- New {public,private}_key_base58check fields.
@meherett
Copy link
Owner

meherett commented Jan 20, 2022

Hi Man, 👍👍 30330b8 I can't merge it now b/c it has git conflicts so I will merge it in the coming weeks. Great work.

- Introduce `DEFAULT_SEMANTIC` to cryptocurrency description;
- Introduce `"base58check"` semantic (for DeSo);
- Introduce `address()` in `HDWallet`;
- Fix xpub/xprv address prefixes for DESOTEST.

The default semantic for Bitcoin is "p2wpkh".
@meherett
Copy link
Owner

Hello @alecov, Can you fix the conflict and create new pull request?

@alecov
Copy link
Author

alecov commented Mar 15, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants