Skip to content

Conversation

@corsacca
Copy link
Member

@corsacca corsacca commented Nov 5, 2025

Adds a network-wide setting to allow subsite administrators to edit users within their own subsites. This complements the capability filters implemented in disciple-tools-theme PR #2789.

Features:

  • New User Permissions tab in Network Admin > Disciple.Tools menu
  • Checkbox to enable/disable subsite admin user editing capability
  • Security documentation explaining restrictions (no super admin editing, no cross-subsite editing)
  • Uses site option 'dt_allow_subsite_admins_edit_users' (disabled by default)

Changes:

  • Created includes/tab-user-permissions.php with settings UI
  • Added tab registration in disciple-tools-multisite.php
  • Added tab navigation and routing in includes/admin-page.php

Security: Feature is disabled by default and requires explicit super admin action to enable.

Adds a network-wide setting to allow subsite administrators to edit users within their own subsites. This complements the capability filters implemented in disciple-tools-theme PR #2789.

Features:
- New User Permissions tab in Network Admin > Disciple.Tools menu
- Checkbox to enable/disable subsite admin user editing capability
- Security documentation explaining restrictions (no super admin editing, no cross-subsite editing)
- Uses site option 'dt_allow_subsite_admins_edit_users' (disabled by default)

Changes:
- Created includes/tab-user-permissions.php with settings UI
- Added tab registration in disciple-tools-multisite.php
- Added tab navigation and routing in includes/admin-page.php

Security: Feature is disabled by default and requires explicit super admin action to enable.
@corsacca
Copy link
Member Author

corsacca commented Nov 5, 2025

@claude this PR fails tests

Add phpcs:ignore comment for NonceVerification.Missing as the nonce is verified in content() method before process_form() is called.
@corsacca corsacca force-pushed the claude/multisite-user-permissions-tab-011CUpZDZQqYzActbuPdEN8G branch from c853160 to 7826324 Compare November 5, 2025 10:22
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.

3 participants