Skip to content

Conversation

@crwood
Copy link
Member

@crwood crwood commented Dec 10, 2021

This PR fixes #405 and provides some additional smaller enhancements intended to improve the safety and user-experience pertaining to creating Recovery Keys -- especially for ZKAPAuthorizer-enabled storage grids. Among them:

  • On ZKAPAuthorizer-enabled grids, attempt to create the rootcap immediately once the first batch of ZKAPs becomes available for spending (rather than waiting until the voucher/batches are fully redeemed or until a user adds a folder) -- and fail loudly with a user-visible error dialog in the event that this fails.
  • Prompt the user to create a Recovery Key after the rootcap has been created and if/when the "Folders" view is visible (i.e., before any folders have been added).
  • Remind the user to create a Recovery Key once -- but only once -- per session in the event that one hasn't been created previously.
  • Add a confirmation buttons ("Cancel", "Save Recovery Key...") to the PasswordDialog to give users clearer or more explicit instructions about how/whether to proceed. (Previously, continuing was triggered by an Enter/Return key press -- while cancelling was triggered by Esc -- which wasn't very discoverable.)
  • Update user-facing language throughout the interface to remove the terms "export" and "import" in the context of handling Recovery Keys (as these were found to be confusing to some users); replace "export" with "create" (or "save" in the context of writing to disk), and "import" with "restore from".

Thereby recording that a Recovery Key has been successfully exported.
Past user testing/feedback has suggested that the terms "import" and
"export" are confusing or unclear in the context of handling Recovery
Keys. Accordingly, replace "export" with "create" (or "save" in the
context of writing to disk) and "import" with "restore from".
@crwood crwood marked this pull request as draft December 10, 2021 15:26
@crwood crwood marked this pull request as ready for review December 12, 2021 23:58
@crwood crwood merged commit 195f435 into master Dec 12, 2021
@crwood crwood deleted the 405.create-recovery-key-prompt branch December 12, 2021 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The "Export Recovery Key" prompt is not displayed automatically on ZKAPAuthorizer-enabled grids

2 participants