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

Update to use latest bitcoinjs #71

Merged
merged 2 commits into from
Sep 9, 2019

Conversation

junderw
Copy link
Contributor

@junderw junderw commented Aug 26, 2019

Also updated mocha because npm audit was reporting some vulnerabilities.

You can confirm package-lock is not malicious by checkout my branch, delete lockfile, then run npm install again, and git should show no diff from what your npm installed.

This BitcoinJS version supports Psbt. So in the future this will probably allow for some sort of multisig functionality? (maybe multiple people using same dojo can store partially signed PSBT on the dojo? idk. I am open to chat if you want)

Yay open source!

Closes #70

@junderw
Copy link
Contributor Author

junderw commented Aug 26, 2019

Most of the changes are package-lock.json

スクリーンショット_2019-08-26_16-10-22

@kenshin-samourai kenshin-samourai added the enhancement New feature or request label Aug 26, 2019
@kenshin-samourai
Copy link
Contributor

i've checked the update quickly. so far so good.

only one comment about the package-lock json: i know that it isn't perfect and that it isn't really the philosophy of the node js community but i feel more comfortable with pinned versions of dependencies in both package.json and package-lock.json.

i'm going to run this update on a test environment to validate that everything is ok.
thanks again!

@kenshin-samourai kenshin-samourai added the WIP Work In Progress label Aug 26, 2019
@kenshin-samourai
Copy link
Contributor

tests ok

@kenshin-samourai kenshin-samourai removed the WIP Work In Progress label Sep 9, 2019
@kenshin-samourai kenshin-samourai merged commit 7c85dac into Samourai-Wallet:develop Sep 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggestion: Use latest bitcoinjs-lib
2 participants