Skip to content

feat: allow admin delegation for System monitoring page - #1187

Open
whoalin1 wants to merge 1 commit into
nextcloud:masterfrom
whoalin1:feat/admin-delegation-system-page
Open

whoalin1 wants to merge 1 commit into
nextcloud:masterfrom
whoalin1:feat/admin-delegation-system-page

Conversation

@whoalin1

@whoalin1 whoalin1 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes #497

The System monitoring page is read-only but required full admin. Implement IDelegatedSettings on AdminSettings (same approach as logreader) so admins can grant the page via Settings → Administration → Delegation without handing out full admin.

getAuthorizedAppConfig() is empty because this UI does not write appconfig.

Test plan

  • As admin: Settings → Administration → Delegation lists System / Monitoring
  • Delegate to a group; member of that group (non-admin) can open the System page
  • Non-delegated users still cannot open it
  • Page still loads for full admins

AI disclosure

This PR was drafted with the help of AI coding assistants (Cursor agents / LLM-based tools), including the description. I am responsible for it and happy to rework anything that doesn't fit.

All commits carry an Assisted-by: Cursor:grok-4.7 trailer.

Assisted-by: Cursor:grok-4.7
Signed-off-by: whoalin1 <whoalin414@gmail.com>
@whoalin1
whoalin1 force-pushed the feat/admin-delegation-system-page branch from 97cecc1 to eae8721 Compare October 9, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin delegation should be possible for system monitoring (page) "systeminfo"

1 participant