-
Notifications
You must be signed in to change notification settings - Fork 309
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
NAS-130139 / 25.04 / Fix unlock children tooltip #10998
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #10998 +/- ##
=======================================
Coverage 82.04% 82.04%
=======================================
Files 1616 1616
Lines 57001 57014 +13
Branches 5894 5897 +3
=======================================
+ Hits 46767 46778 +11
- Misses 10234 10236 +2 ☔ View full report in Codecov by Sentry. |
@@ -6,7 +6,8 @@ export const helptextUnlock = { | |||
unlock_key_file_placeholder: T('Unlock with Key file'), | |||
unlock_key_file_tooltip: T('Use an exported encryption key file to unlock datasets.'), | |||
unlock_children_placeholder: T('Unlock Child Encrypted Roots'), | |||
unlock_children_tooltip: T('Also unlock any encrypted dataset stored within this dataset.'), | |||
unlock_children_tooltip: T('Also unlock any encrypted dataset stored within this dataset. \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's change first line to "Also unlock any separate encryption roots that are children of this dataset".
This PR has been merged and conversations have been locked. |
Changes:
Updated the tooltip
Testing:
Please follow replication steps in the ticket.
Expected result: tooltip is displayed correctly