Ledger MASP Integration #1575
Draft
Ledger MASP Integration #1575
+1,240
−331
Conversation
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
jurevans
added
App: Namadillo
App: Extension
Ledger HW Wallet
* feat: bump to latest Ledger package * feat: begin hooking up shielded keys import * feat: split proof-gen and viewing key calls, fix paths * fix: fix tests * feat: stealing Mateusz code, show payment address in confirmation * feat: validate semver of installed app for Zip32 * feat: begin implementing import approval steps
* feat: ledger shielded accounts * fix: prettier fmt
mateuszjasiuk
force-pushed
the
feat/ledger-masp-integration-branch
branch
from
c02399f to
af9a137
Compare
* feat: support ledger masp tx wip * feat: unshielding working * chore: cleanup * feat: ledger masp with disposable signer * feat: disposable signer for shielded transfers using ledger * feat: proper flow for ledger txs from masp source
…r device (#1572) * feat: disable submit button for masp tx using disconnected ledger * fix: do not check for conencted ledger while tx is submitting
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR combines all work on Ledger MASP integration into a single branch ahead of actually merging to
mainand releasing keychain. Once the Ledger Namada App is released on Ledger Live, we can do a final review, merge, and release.Original PRs: