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.

Sign up for GitHub

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

Jump to bottom

feat: add admin toggle for e2ee calls #14098

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

feat: add admin toggle for e2ee calls #14098

wants to merge 5 commits into from

Conversation

Antreesy
Copy link
Contributor

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
image image

🚧 Tasks

  • ...

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

🛠️ API Checklist

🚧 Tasks

  • ...

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen mentioned this pull request Jan 13, 2025
Closed
14 tasks
@Antreesy Antreesy force-pushed the feat/14005/admin-toggle branch from 4ef25fd to 3d10280 Compare January 13, 2025 13:09
@nickvergessen
docs: Document the app config setting
f48b2f9 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the feat/14005/admin-toggle branch from 3d10280 to be36816 Compare January 15, 2025 11:50
@fancycode
Copy link
Member

@nickvergessen I have updated #14005 to check the feature / app config to enable e2ee.

nickvergessen and others added 4 commits January 15, 2025 16:38
@nickvergessen @Antreesy
feat(e2ee-calls): Add a feature capability and config capability for …
63368a1 
…end-to-end encrypted calls

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @Antreesy
feat(setupcheck): Show an admin check if the feature is not supported…
4d5379a 
… by the HPB

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @Antreesy
fix(mobile-clients): Require supported mobile clients when E2EE calls…
292aa49 
… are enforced

Signed-off-by: Joas Schilling <coding@schilljs.com>
@Antreesy
fix(setting): add admin toggle for e2ee calls
157ba59 
- Show setting only with HPB and adjust condition for being able to enable it

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the feat/14005/admin-toggle branch from 1c2c3e4 to 157ba59 Compare January 15, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fancycode fancycode fancycode left review comments

@nickvergessen nickvergessen nickvergessen requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Antreesy @fancycode @nickvergessen