Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

add params to import xpriv #259

Merged
merged 7 commits into from
May 9, 2016
Merged

add params to import xpriv #259

merged 7 commits into from
May 9, 2016

Conversation

matiu
Copy link
Collaborator

@matiu matiu commented May 5, 2016

No description provided.

@coveralls
Copy link

coveralls commented May 5, 2016

Coverage Status

Coverage increased (+0.03%) to 94.555% when pulling b001c1b on matiu:feat/import-xpriv into 7f15c05 on bitpay:master.

@@ -88,9 +88,20 @@ Credentials.createWithMnemonic = function(network, passphrase, language, account
return x;
};

Credentials.fromExtendedPrivateKey = function(xPrivKey) {
Credentials.fromExtendedPrivateKey = function(xPrivKey, derivationStrategy, addressType) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we also include account?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes!

@matiu
Copy link
Collaborator Author

matiu commented May 6, 2016

Updated. Adds account, adds params also to API. endpoint and fix tests and documentation

@coveralls
Copy link

coveralls commented May 6, 2016

Coverage Status

Coverage increased (+0.04%) to 94.572% when pulling d4ba7ce on matiu:feat/import-xpriv into 7f15c05 on bitpay:master.

@coveralls
Copy link

coveralls commented May 6, 2016

Coverage Status

Coverage increased (+0.04%) to 94.572% when pulling 6813a7a on matiu:feat/import-xpriv into 7f15c05 on bitpay:master.

@matiu
Copy link
Collaborator Author

matiu commented May 6, 2016

updated.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 94.565% when pulling 763aa61 on matiu:feat/import-xpriv into 7f15c05 on bitpay:master.

@isocolsky isocolsky merged commit 8fdc92a into bitpay:master May 9, 2016
sparklecoin pushed a commit to sparklecoin/bitcore-wallet-client that referenced this pull request Nov 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants