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

[GUI] Export the descriptor into a file #1547

Open
nondiremanuel opened this issue Jan 15, 2025 · 10 comments
Open

[GUI] Export the descriptor into a file #1547

nondiremanuel opened this issue Jan 15, 2025 · 10 comments
Assignees
Labels
Enhancement Improving an existing functionality GUI gui related

Comments

@nondiremanuel
Copy link
Collaborator

It should now be possible to export an element into a file. We should use such feature for things like the descriptor, PSBTs, labels ...

We may start by letting the user export the descriptor into a file.

After that we should implement the possibility to import such file in the "Add an existing wallet" flow (we can deal this in a separate issue).

@nondiremanuel nondiremanuel added GUI gui related Enhancement Improving an existing functionality labels Jan 15, 2025
@nondiremanuel nondiremanuel moved this to Todo in Liana General Jan 15, 2025
@pythcoiner
Copy link
Collaborator

do you want this featur to replace the actual "copy descriptor" one or you prefer keep both?

@edouardparis
Copy link
Member

I think it is better to have both.

@kloaec
Copy link
Collaborator

kloaec commented Jan 17, 2025

I think it is better to have both.

Is there any use of copying a descriptor that would impact the majority of our users?
If yes, we can keep both. If not, i don't think we should keep it.

@nondiremanuel
Copy link
Collaborator Author

We don't have much information on the current way of storing the descriptor of our users, so we can try to guess. My guess is that they currently copy it to paste it in a file, so probably it would be redundant to have both. I would be keen on substituting the "copy" button for this reason, unless I'm missing some use case.

@edouardparis
Copy link
Member

The descriptor is displayed, user should be able to copy it and for its usage I was thinking copy paste it to a terminal etc.

@nondiremanuel
Copy link
Collaborator Author

The descriptor is displayed, user should be able to copy it and for its usage I was thinking copy paste it to a terminal etc.

By downloading the file you can still open it and copy the descriptor, so this would not be disallowed (especially since people that want to use a terminal surely know how to do it). The disadvantage of course is that you need more steps.
On the other hand, having both the "Copy" and the "Download" button could be messy for non-technical users, so both solutions have their own trade-offs.

I personally lean towards avoiding confusion for average users over facilitating terminal edge-cases, but it's not a huge deal either way.

@nondiremanuel nondiremanuel moved this from Todo to In Progress in Liana General Jan 22, 2025
@thibistaken
Copy link

I can see value in both approaches, personally:

Copy/Paste: it's quick and easy, works well for password managers & avoids unnecessary file downloads (PDFs).

File export: it enables storage on private drives/servers, saves time vs. manual file generation, and can include helpful metadata (timestamp, wallet name, Liana branding & additional context unavailable in plain text).

Both options serve different use cases, so supporting both would provide the best user experience IMO.

@nondiremanuel
Copy link
Collaborator Author

I can see value in both approaches, personally:

Copy/Paste: it's quick and easy, works well for password managers & avoids unnecessary file downloads (PDFs).

File export: it enables storage on private drives/servers, saves time vs. manual file generation, and can include helpful metadata (timestamp, wallet name, Liana branding & additional context unavailable in plain text).

Both options serve different use cases, so supporting both would provide the best user experience IMO.

Thanks for your feedback.
Interesting consideration regarding the password managers use case. Shall we consider saving the descriptor in a password manager a good practice (and so facilitate it)? I'm not sure given the type of data we are talking about.

By the way, we are currently working on building the structure of an entire wallet backup file which would include the descriptor but also additional data of the wallet which should improve the backup & restore process and make the descriptor backup not necessary anymore (even if we will still guarantee backward compatibility).

@thibistaken
Copy link

thibistaken commented Feb 19, 2025 via email

@nondiremanuel
Copy link
Collaborator Author

What do you think would be the risk storing this data in a password manager? Descriptor is public data about the wallet in this context, right?

I was not specifically talking about security (which afaict could be more or less equivalent to have an encrypted backup on one's cloud or so), but on later availability, for the user himself and/or possible heirs. I didn't think about it much, but it's an interesting discussion to have.

Instinctively I feel that having a file is more effective in conveying that this is your wallet data which you need to store than a long string to copy/paste.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improving an existing functionality GUI gui related
Projects
Status: In Progress
Development

No branches or pull requests

5 participants