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.

Sign up for GitHub

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

Jump to bottom

Launch Bitcoin-Qt before importing Descriptors #128

Closed
BenWestgate opened this issue May 15, 2024 · 1 comment · Fixed by #161
Closed

Launch Bitcoin-Qt before importing Descriptors #128

BenWestgate opened this issue May 15, 2024 · 1 comment · Fixed by #161
Assignees
@BenWestgate
Labels
enhancement New feature or request good first issue Good for newcomers priority: low Issues raised by only myself
Milestone
L2

Comments

@BenWestgate
Copy link
Owner

Is your feature request related to a problem? Please describe.
No visual progress is displayed during import descriptors although rescanning the blockchain for transactions is a potentially slow operation.

Describe the solution you'd like
Stop bitcoind and start bitcoin-qt before importing descriptors in bails-wallet

Describe alternatives you've considered
Running bitcoin-qt the entire time bails-wallet is running. This provides an opportunity for distraction and consumes more resources making the GUI potentially laggy.
@BenWestgate BenWestgate self-assigned this May 15, 2024
@BenWestgate BenWestgate added enhancement New feature or request good first issue Good for newcomers priority: low Issues raised by only myself labels May 15, 2024
@BenWestgate BenWestgate modified the milestones: L1 (BETA), L2 May 15, 2024
@BenWestgate BenWestgate linked a pull request May 26, 2024 that will close this issue
Improve Verification #161
Merged
@BenWestgate
Copy link
Owner Author

This is solved by #161 which uses bitcoin-qt -min -server to handle the rpc commands.

This is best since restarting Bitcoin before the headers download causes a complete loss of progress. As well as a complete flushing of caches if they get even further along.

@BenWestgate BenWestgate mentioned this issue May 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BenWestgate BenWestgate

Labels
enhancement New feature or request good first issue Good for newcomers priority: low Issues raised by only myself
Projects
None yet
Milestone
L2
Development

Successfully merging a pull request may close this issue.

Improve Verification BenWestgate/Bails
1 participant
@BenWestgate