Skip to content

πŸ‘‘ feat(owners): Allow to promote/demote users to/from owners - #18924

Open
nickvergessen wants to merge 5 commits into
mainfrom
feat/noid/owner-promotion-and-demotion
Open

πŸ‘‘ feat(owners): Allow to promote/demote users to/from owners#18924
nickvergessen wants to merge 5 commits into
mainfrom
feat/noid/owner-promotion-and-demotion

Conversation

@nickvergessen

@nickvergessen nickvergessen commented Aug 7, 2026

Copy link
Copy Markdown
Member

β˜‘οΈ Resolves

Decisions:

  • Only owners can promote to owners
  • Owners are not mandatory, only 1 moderator remains mandatory
  • Self-demotion to moderator is allowed
  • Removing owners directly is not possible, it's needed to demote them first

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

πŸ–ŒοΈ UI Checklist

πŸ–ΌοΈ Screenshots / Screencasts

User (with extended permissions) Moderator Owner Participant list
Bildschirmfoto vom 2026-08-07 14-40-58 Bildschirmfoto vom 2026-08-07 14-41-31 Bildschirmfoto vom 2026-08-07 14-41-42 Bildschirmfoto vom 2026-08-07 15-02-40

🚧 Tasks

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • 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

🏁 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

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen force-pushed the feat/noid/owner-promotion-and-demotion branch from 03dc5cf to b66b8fe Compare August 7, 2026 14:06

@jancborchardt jancborchardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sounds good, but the wording would be better as "participant" instead of "user" as we use that in the tab header already.

And the 2 titles in src/components/RightSidebar/Participants/ParticipantItem.vue should probably start with a capital letter. It looked fine lowercase when they were in parentheses behind the username, but in a standalone title it looks better properly cased.

@github-project-automation github-project-automation Bot moved this to πŸ—οΈ In progress in πŸ’¬ Talk team Aug 10, 2026
@jancborchardt jancborchardt moved this to πŸ—οΈ At engineering in πŸ– Design team Aug 10, 2026
@Antreesy
Antreesy requested a review from jancborchardt August 10, 2026 15:38
@Antreesy

Copy link
Copy Markdown
Contributor

Addressed design commits, checked
From frontend side - good to go

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@Antreesy
Antreesy force-pushed the feat/noid/owner-promotion-and-demotion branch from 8ff3afb to b33d964 Compare August 10, 2026 15:53
@SystemKeeper
SystemKeeper self-requested a review August 10, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits enhancement feature: api πŸ› οΈ OCS API for conversations, chats and participants feature: conversations πŸ‘₯ feature: frontend πŸ–ŒοΈ "Web UI" client

Projects

Status: πŸ—οΈ In progress
Status: πŸ—οΈ At engineering

Development

Successfully merging this pull request may close these issues.

4 participants