-
Notifications
You must be signed in to change notification settings - Fork 63
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
Upgrade Bitcoin app version if above minimum but below Taproot minimum #1135
Comments
how do you see this? call ledger_installer gui as an external process? |
No, i think the GUI should just have a button letting the user upgrade in case they choose to use Taproot but their device's Bitcoin app is too old. This would just call the library's function directly. |
open a modal to show hint (accept on device) and progress (bubling sardine?)? |
I'm not sure yet how it should be integrated, but i think it should be minimal. For instance if we choose to have the upgrade button inside the signing devices list item, then opening a pop-up would be a pop-up on top of a pop-up which might not be ideal. Maybe when you start tackling this, start by sketching out the idea and propose it here before moving on with the implementation? |
I've started look at this, i think we should at least have the feature in 3 location:
The way i think to achieve this is to grey-out (disable) the signing device selection button and add inside it an " Upgrade my ledger" button that will trigger the upgrade process while keeping the modal opened: Points to consider
|
I was wondering why it would be necessary after the installer step, but it may be the case that a participant who shared his xpub may not have the appropriate version. In this case we should indeed have a version check before registering the descriptor and instruct the user to upgrade if the version is too old. But why having it at signing time? You may only ever sign with a device upon which was registered the descriptor, so having the check there seem sufficient? |
Sure, we can do it but it's not a blocker. Especially if it turns out we need to adjust a couple things.
Probably, yes. We can never ensure the user won't interrupt the upgrade though.
Ideally we'd also show the progress on the UI but i can understand how it can be difficult.
Why not, but let's keep it simple for now. (Also this conflicts with your second statement of being stuck on the modal.) |
It's not clear for the user when they should click on the button nor is it technically correct because of the size restriction. Instead what do you think of:
|
In the case an user have broken his ledger, replace by a new one, import his private key into, as Liana store the PoR, i think it will be detected as registered. So in this case user can end up to signing step w/o Liana told him the device is not registered. |
user can only interrupt on hardware side and i think we can detect it and then allow user to leave the modal
I think a progress bar is doable, I don't think difficult but not sure if it worth the time |
They can just close the software altogether. Preventing to close the modal is just to prevent a simple mistake imo (and might be worth it), but it's not possible to protect the user from themselves in this case. |
it's not exactly what you have drafted, but what do you think about this (modulo the color theme that should been adjusted for the button at least): (i do not see an easy way to make the yellow banner 'attached' to the device button, so I put it inside the button, as if below,the user can be confused if there is 2 following ledger in the list) |
The last one looks good to me (besides using a white font color for the Upgrade button).
…-------- Original Message --------
On 7/24/24 5:03 AM, pythcoiner wrote:
or like this:
[image.png (view on web)](https://github.com/user-attachments/assets/df4950b0-fe3f-4d96-aa73-43098f6fb022)
—
Reply to this email directly, [view it on GitHub](#1135 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AFLK3F4JGJFQSWGMHSISFLDZN4KOTAVCNFSM6AAAAABJZYBWO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBWG43TCMJYG4).
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
One thing that could be worth considering is that this modal could be part of the UX refactoring. |
Enough people have been testing https://github.com/darosior/ledger_installer that i feel comfortable starting to integrate pieces into Liana. A good small first step would be to make it possible for Ledger users with a Bitcoin app whose version is below the version which introduced Taproot support to upgrade to the latest Bitcoin app. Note however those users must already have a Liana-compatible app (above our minimum supported version) otherwise the update of the app would also necessitates a firmware upgrade, which is not implemented at the moment (and not the best candidate for a first step).
The text was updated successfully, but these errors were encountered: