-
Notifications
You must be signed in to change notification settings - Fork 40
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
UI Only AU Snapshot Import Flow #421
base: main
Are you sure you want to change the base?
Conversation
Concept ack |
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.
Doesn't it make more sense to (also) have this part of the onboarding workflow?nvm, I think we shouldn't do this (maybe later)- Also, having it at
Connection
menu would not be something where I would be looking at to find this feature. Maybe it needs anAdvanced
section? - A UI for an incorrect snapshot and/or failure of importing the snapshot is missing
id: gotoSnapshot | ||
Layout.fillWidth: true | ||
header: qsTr("Import snapshot") | ||
description: qsTr("Speed up the initial download") |
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.
as you said correctly in the import snapshot dialog, it doesn't speed up the download itself.
So this might be misleading.
I think it's better to rephrase it, something like this might be more appropriate Start using right away and do initial synchronization in the background
or maybe something short like Do initial synchronization in the background
@@ -0,0 +1,35 @@ | |||
// Copyright (c) 2024 - PresentThe Bitcoin Core developers |
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.
// Copyright (c) 2024 - PresentThe Bitcoin Core developers | |
// Copyright (c) 2024-present The Bitcoin Core developers |
nit
and at other files you created it has 2023 instead of 2024
Introducing the UI flow to be able to import a AssumeUTXO (AU) snapshot into the Bitcoin Core App.
The user will be able to load an AU snapshot from the "Connection Settings" page as designed here
*This is a UI only commit no actual functionality has been coded. Wiring to actually import a AU snapshot will happen in a future PR
Ubuntu 22.04 Screenshots