-
Notifications
You must be signed in to change notification settings - Fork 58
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] Add Electrum backend option to Installer & update Settings #1223
Comments
For the installer, we currently have this page: We could either:
Perhaps we should opt for fewer pages/clicks, in which case (1) would be better. |
I like the idea of having the 3 options in columns, each in a card with a title that needs to be clicked on to collapse and show the text and the button next. |
Intuitively (but probably I'm missing something), I would expect to have a three steps choice:
The choice branching would be something like:
In this case, electrum would be an option in the following page in which you put your IP and so on: The reason for that is that people could be confused by the fact of having two options to link to their own node at that stage. What am I missing? |
Thanks for the suggestions. It makes sense to consider how these options and the flow will link with the Liana Connect option.
I tend to think of "node" as Bitcoin Core, so I think we could change this to "or your own backend (Bitcoin node or Electrum server)".
Changing the existing wording would be the simplest thing, but having the three cards would also be nice.
If we stick with the single "manage your own backend" option, then I think we could have another radio button at the top of this page to select the backend type ("Bitcoin node" or "Electrum" server) and then the contents below would change accordingly. The title of the page would be "Set up connection to the Bitcoin backend". If we have three separate options, then we would just need to add a new page for the Electrum connection and the existing "Set up connection to Bitcoin full node" page would stay the same. |
I think your wording is more precise. I added a proposal on this step in #970, so that we can keep the discussion about it there.
I tend to think that the main criteria of choice is: do you already have a node/server to connect to or do you want us to manage (and download) one for you? That's why I personally lean a little bit more towards changing the wording and adding the Electrum option in the following screen. But let's see if we can find an agreement or other arguments.
Yes, that's very similar to how I imagined it.
So, to recap, the options at the moment are:
In terms of effort required, do you see any major difference between the two? |
Perhaps slightly simpler to have a separate page for setting up Electrum, but either way should be similar effort. |
After clarifying the previous step in the relative PR, among the two options for showing Electrum I would personally go with the first:
I would see the wording of the first two options (what now is the "Bitcoin Node Management" step - we should consider changing this title as well imho) something like:
The wording has to be reviewed and validated of course, but I think it makes it simpler and less redundant. If the user selects 1, there should be two options: as @darosior pointed out:
Other opinions or comments? |
Regarding Electrum, the user could in principle enter any valid URL, e.g. a public instance. We could add some text to recommend only connecting to a self-hosted Electrum server for privacy reasons. |
definitely needs to be self hosted. It should be very clear that it's not just a recommendation, but important |
819eb92 gui(settings): allow to change node type (Michael Mallan) 2381227 gui(settings): view & edit Electrum settings (Michael Mallan) b570039 gui(settings): rename Bitcoin Core to Node (Michael Mallan) db20ae4 gui(installer): reduce empty space height (Michael Mallan) 0993905 gui(installer): update wording to include Electrum (Michael Mallan) f40af57 gui(installer): split long string and run cargo fmt (Michael Mallan) 0f09be1 gui(installer): don't change values while waiting for ping result (Michael Mallan) c93aa88 gui(installer): add electrum node option (Michael Mallan) 341e446 gui(installer): allow for different node types (Michael Mallan) 83172c7 gui(installer): add general node definition (Michael Mallan) 046b54e gui(installer): define bitcoind from general node struct (Michael Mallan) c5d9d00 gui: move bitcoind to new node module (Michael Mallan) 4536eff gui(installer): extract logic for try ping bitcoind (Michael Mallan) ef44cf3 gui(installer): add module for node step (Michael Mallan) f74f071 gui: upgrade liana dependency (jp1ac4) Pull request description: This is for #1223. For now, it's possible to edit the node's settings but not to change node type. Remaining tasks: - [x] Revert Cargo.toml once #1222 is merged. - [x] Update wording as per #1223 (comment). ACKs for top commit: pythcoiner: ACK 819eb92 Tree-SHA512: 362a14d32c2e13ba286d252d9f8a1106d63e5c40198776653b0623b433435329663126307e17da017fdbbd8a8ad273b703cc3ba54fd13fa5a0afd7dd9179089a
Done in #1241. |
Following #1222, we will need to add an Electrum option in the Installer and update the Settings page accordingly.
The text was updated successfully, but these errors were encountered: