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

[stable29] fix(settings): Clarify peculiarities of enabling encryption #50596

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jan 31, 2025

Summary

  • Clarify that enabling server side encryption will not encrypt existing files but only new or changed files.
  • Clarify that server side encryption can only be disabled using OCC
  • Ensure there is accessible information of encryption state (disabled input will not be announced so make it aria-disabled instead)
  • Make warning more prominent by moving it into a dialog

- Clarify that enabling server side encryption will not encrypt
  existing files but only new or changed files.
- Clarify that server side encryption can only be disabled using OCC
- Ensure there is accessible information of encryption state (`disabled`
  input will not be announced so make it `aria-disabled` instead)
- Make warning more prominent by moving it into a dialog

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux self-assigned this Jan 31, 2025
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@susnux susnux enabled auto-merge January 31, 2025 17:46
@susnux susnux merged commit a0d87fb into stable29 Feb 3, 2025
113 checks passed
@susnux susnux deleted the backport/50424/stable29 branch February 3, 2025 10:01
@Altahrim Altahrim mentioned this pull request Feb 5, 2025
5 tasks
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