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: reload home if wallet is not syncing #1437

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

jp1ac4
Copy link
Collaborator

@jp1ac4 jp1ac4 commented Nov 5, 2024

Following #1386, this PR will show past payments on the home page while the blockchain is still syncing. Currently, these are not loaded until the wallet has finished syncing.

This change won't apply if the wallet itself is still syncing. The reason is that this is not expected to take so long and so it's better to wait for the updated data. Furthermore, the DB may be locked if the poller is running, in which case we wouldn't be able to load past transactions until the wallet has synced anyway. The home page reloads automatically as soon as the wallet finishes syncing.

@jp1ac4 jp1ac4 self-assigned this Nov 5, 2024
@nondiremanuel nondiremanuel added GUI gui related Enhancement Improving an existing functionality labels Nov 7, 2024
@jp1ac4 jp1ac4 force-pushed the gui-reload-home-condition branch from 18dfe6c to 71ad55e Compare November 15, 2024 13:47
@jp1ac4
Copy link
Collaborator Author

jp1ac4 commented Nov 15, 2024

Rebased on master.

Copy link
Member

@edouardparis edouardparis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 71ad55e

@edouardparis edouardparis merged commit 27dce54 into wizardsardine:master Nov 18, 2024
24 checks passed
@jp1ac4 jp1ac4 deleted the gui-reload-home-condition branch November 20, 2024 14:55
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: Done
Development

Successfully merging this pull request may close these issues.

3 participants