-
Notifications
You must be signed in to change notification settings - Fork 492
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
Fix .txn file import for ColdCards #11662
Conversation
catch (FormatException ex) | ||
{ | ||
throw new FormatException("An error occurred while loading the PSBT file.", ex); | ||
} |
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.
Wasabi couldn't broadcast the .txn
file, because here we throw an exception, instead of letting it run to the "fallback" catch block.
If the fallback fails too, we are catching the exception outside and an error dialog will be shown to the user.
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.
tACK
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.
utACK
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.
utACK
We tested this in the office with @Szpoti and I'm not aware of anyone having a CC at home, so merging. |
* add fallbacks when PSBT loading fails * add .txn file support to FileDialogs * Typo * add log messages + remove unused usings --------- Co-authored-by: yahiheb <52379387+yahiheb@users.noreply.github.com> (cherry picked from commit 362b7de)
Addresses: #9856 (comment)
Might fix completely: #9856
When the user sets this settings on ColdCard (
settings>delete psbts>normal/delete)
todelete
,ColdCard deletes the PSBT files from the SD card.
What's left is a signed
.txn
file.Wasabi almost supported this, but because we throw this ex:
throw new FormatException("An error occurred while loading the PSBT file.", ex);
the fallback mechanism didn't have a chance to run.
With this PR I was able to broadcast the .txn file properly.