cryptomator/cryptomator

[1.5.0] Ad existing vault: point user to recovery key

Open

#1,028 opened on Dec 22, 2019

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Java (10,325 stars) (1,068 forks)batch import
good first issuemisc:gui

Description

Summary

After adding a vault (no matter if new or existing), users should be actively made aware of the recovery key

Motivation

When creating a new vault, the user is actively made aware of the recovery key (and must confirm that he saved it). But if he just ads an existing vault, or if he migrates one, there's no popup. You have to remind on your own to look up the key in the vault options and save it. As the recovery key might be relevant, there should be a note in every scenarios that adds a vault to cryptomator.

Contributor guide

Tech stack
java
Domain
desktopsecurity
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JavaJavaFXCryptomator vault workflow
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
The issue asks for a popup or notification when adding an existing vault. Start by finding the code that handles adding a new vault and displays the recovery key confirmation (likely in the UI controller for vault creation). Then, locate the code path for adding an existing vault and add a similar recovery key reminder. The exact files may include VaultAddController.java or VaultOptionsController.java. Check for existing UI components that can be reused.
[1.5.0] Ad existing vault: point user to recovery key · cryptomator/cryptomator#1028 | Good First Issue