Skip to content

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 <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@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.

4 participants