-
Notifications
You must be signed in to change notification settings - Fork 32
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
webusb and no verify xpub export #24
Merged
bucko13
merged 16 commits into
unchained-capital:master
from
humanumbrella:webusb-and-no-verify
Jun 25, 2020
Merged
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
f3dae91
code styling, fix errors and typos
humanumbrella fdd68cf
eslint-disable `max-lines` in `eslintrc.json`
humanumbrella 0b0fa67
include @ledgerhq/transport-webusb and disable verify:true on pubkey …
humanumbrella 8d0cb47
bump hw-app-btc version
humanumbrella b3ea93e
code review feedback - not async function
humanumbrella ba79e8f
bump all ledger dependencies
humanumbrella 61a755e
Explicitly remove U2F and code review
humanumbrella e516292
update package-lock.json
humanumbrella c3d7270
remove unnecessary async's
humanumbrella 43e24bc
hw-app-btc version that includes `serializeTransactionOutputs`
humanumbrella f380389
bring hw-transport-u2f back for Firefox fallback option
humanumbrella 1ff9a80
comment out messages in pubkey export tests for now
humanumbrella b6b4ead
comment out messages in xpub export
humanumbrella 2449f11
images unused in this release
humanumbrella 8cd4d6a
cleanup and remove commented code
humanumbrella 7a5b6bd
remove commented code
humanumbrella File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Wow, we're not using the images at all anymore? I guess we don't use them or have them yet for signing?
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.
we were only showing images for xpubs/pubkeys at the moment
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.
See also unchained-capital/caravan#112