feat(aliases): add admin setting to disable alias creation#12764
feat(aliases): add admin setting to disable alias creation#12764bromiesTM wants to merge 1 commit intonextcloud:mainfrom
Conversation
|
Thank you for sending a pull request |
|
Please point your agent to https://github.com/nextcloud/mail/blob/main/AGENTS.md#git-workflow in case you didn't open it from this repo's root. Agentic AI has to be transparent in commit messages. |
93543cf to
eea743b
Compare
jancborchardt
left a comment
There was a problem hiding this comment.
Just giving the design-ok for the record. This is useful for providers who want to prevent alias creation or manage it elsewhere.
@bromiesTM for future pull requests please also always include the use-case or the issue the pull request tackles. That helps with deciding how to go ahead with it. Thank you! :)
a7d30b0 to
e5a83f4
Compare
Introduces an `allow_new_mail_aliases` app config flag (default: yes) that lets administrators prevent users from creating new mail aliases. - Backend: guard in AliasesService::create() throws ClientException when disabled - Admin UI: toggle switch in AdminSettings, mirroring the existing "allow new mail accounts" setting - Frontend: hides the "Add alias" button when disabled - Exposed via PageController initial state and storable via occ config:app:set Extends existing unit tests for AliasesService, AdminSettings, and PageController to cover the new setting. AI-assisted: Claude Code (Claude Sonnet 4.6) Co-Authored-By: Kai Henseler <kai.henseler@strato.de> Signed-off-by: Matthias Sauer <sauerm@strato.de>
e5a83f4 to
aca5dd0
Compare
|
What about the Alias to S/MIME certificate mapping section? It will still be shown when aliases are disabled by the admin. Should it be hidden? |
|
Tested and works otherwise. Thank you! |
Introduces an
allow_new_mail_aliasesapp config flag (default: yes) that lets administrators prevent users from creating new mail aliases. This is useful for providers who want to prevent alias creation or manage it elsewhere.Extends existing unit tests for AliasesService, AdminSettings, and PageController to cover the new setting.