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

Offline Psbt Workflow compatible #12288

Merged
merged 5 commits into from
Jan 23, 2024
Merged

Conversation

Whem
Copy link
Collaborator

@Whem Whem commented Jan 19, 2024

Introduction

Not all Hardware Wallets support offline PSBT via SD card. That is why this PR was created to be able to regulate this in the future.

#Solution
To be able to prepare for this task, I had to go through the datasheet on every device. It is also possible to sign the transaction with PSBT on the other HardwareWallets, but for that, you have to use another external application, which makes PSBT meaningless, because the Hardware Wallet must not be connected to the computer. In this case, it is enough to accept the transaction directly on HarwareWallet while it is connected to the computer.

#Results

  • Ledgers
    • Not supported
  • Coldcard
    • Supported
  • Jade
    • Supported the Air gapped PSBT but recently has not implemented in WW
  • Trezors
    • Trezor One
      • Not supported
    • Trezor T
      • Supported

@Whem Whem self-assigned this Jan 19, 2024
@Whem Whem requested a review from molnard January 19, 2024 15:00
molnard
molnard previously approved these changes Jan 22, 2024
Copy link
Collaborator

@molnard molnard left a comment

Choose a reason for hiding this comment

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

cACK, what is the next step?

@Whem
Copy link
Collaborator Author

Whem commented Jan 22, 2024

cACK, what is the next step?

We need to merge it and pass it to UI team. They need to handle this Property in Settings option

Co-authored-by: yahiheb <52379387+yahiheb@users.noreply.github.com>
Copy link
Collaborator

@molnard molnard left a comment

Choose a reason for hiding this comment

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

ACK

@molnard molnard merged commit e0dd89f into WalletWasabi:master Jan 23, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants