Skip to content

Improved GUI to clarify the passphrase field#2208

Merged
m3nu merged 4 commits intoborgbase:masterfrom
VandalByte:dev-passphrase-issue
Feb 20, 2025
Merged

Improved GUI to clarify the passphrase field#2208
m3nu merged 4 commits intoborgbase:masterfrom
VandalByte:dev-passphrase-issue

Conversation

@VandalByte
Copy link
Collaborator

Description

This PR adds more clarity to passphrase field of "Add new repository" and "Add existing repository" sections.

Changes made:

  • Added placeholder text in passphrase field for better clarity for the users.
  • Added a NOTE section to notify the importance of keeping the entered passphrase safe.
  • UI of repo_add.ui has been modified.
  • No existing functionality has been changed.

Related Issue

Resolves #2194

Motivation and Context

Several users reported confusion regarding the passphrase field in the "Add new repository" of Vorta and requested an improved UI with clearer information.

How Has This Been Tested?

  • Tested the UI by running the build

Screenshots:

Screenshot 2025-02-19 233714

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  •  I have read the CONTRIBUTING guide.
  •  My code follows the code style of this project.
  •  My change requires a change to the documentation.
  •  I have updated the documentation accordingly.
  •  I have added tests to cover my changes.
  •  All new and existing tests passed.

I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.

@m3nu m3nu merged commit 93ef64c into borgbase:master Feb 20, 2025
7 checks passed
@VandalByte VandalByte deleted the dev-passphrase-issue branch February 20, 2025 15:32
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.

Please improve the GUI to clarify the "passphrase" field: is it for BorgBase? for the SSH key? for encryption?

2 participants