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] Automatic wallet backup after upgrade to HD #1431

Merged
merged 1 commit into from
Mar 23, 2020

Conversation

random-zebra
Copy link

@random-zebra random-zebra commented Mar 19, 2020

Based on

Offer the option to directly backup the wallet after upgrade-to-HD (and successive re-lock) is completed.

The message on the initial dialog is changed (from "Note that you will need to MAKE A NEW BACKUP.."):

backup1

If the user clicks "OK", and the wallet is locked, the unlock dialog is directly shown (as per #1387)

backup2

After the upgrade, the client directly presents this dialog:

backup3

If the user clicks "OK", then the open-file dialog is presented, so the user can select the filename for the backup file and click "Save" completing the flow.
If the user clicks "CANCEL" (or dismisses the open-file dialog later), a snackbar popup warns the user, telling to remember to backup asap.

@random-zebra
Copy link
Author

Rebased, now that #1387 is merged.

Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

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

Tested ACK 53cfbee.

Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 53cfbee

@random-zebra
Copy link
Author

merging...

@random-zebra random-zebra merged commit 01651b6 into PIVX-Project:master Mar 23, 2020
@random-zebra random-zebra deleted the 2020_upgrade_backup branch September 24, 2020 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants