-
Notifications
You must be signed in to change notification settings - Fork 7
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
Improvements #58
Improvements #58
Conversation
657f79b
to
2bb627e
Compare
00c7529
to
852c117
Compare
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.
Great stuff. Would be nice to get this in a merge-ready state.
CodeScannerView(codeTypes: [.qr], completion: model.handleScan) | ||
} | ||
.alert(item: $model.scanQRError) { error in | ||
Alert(title: Text("Import PSBT error"), |
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.
This title is wrong when scanning an address.
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.
fixed in 888f99f
Not sure if this is related to this PR, but when I tap on the address it gets the strike-through. It should only happen if I tap on the check mark. Let's drop the "Address" header at the top, so "Scan to check" is at the top. It already says "Addresses" in the tab bar below, so it's redundant. |
I'll fix it in a separate PR, #62 |
ab1b861
to
debe292
Compare
debe292
to
7f6917e
Compare
Next bulletpoins moved into another PR
Explain why sign button is disabled #30possible fixes
Crash when clicking Load PSBT #47[Feature] Add "Open with..." feature to import files directly from Specter/AirDrop #15