Skip to content

feat: Let modules inject admin tabs when editing a user group - #539

Merged
hellopablo merged 1 commit into
developfrom
feature/group-admin-tabs-develop
Sep 9, 2026
Merged

feat: Let modules inject admin tabs when editing a user group#539
hellopablo merged 1 commit into
developfrom
feature/group-admin-tabs-develop

Conversation

@hellopablo

Copy link
Copy Markdown
Member

Summary

  • Mirrors the group-tab injector from feature/pre-new-admin onto develop.
  • Adapts the implementation to the namespaced src/Admin/Controller/Groups.php controller and the current permission service.
  • Removes the hard-coded legacy 2FA view and lets modules validate, render, and persist group-specific tabs.

Original feature/pre-new-admin PR: #538.

Test plan

  • PHP syntax checks pass for all changed PHP files
  • PHPUnit: 148 tests, 300 assertions
  • Edit a group and confirm Basic Details, Password, Permissions, and injected MFA tabs render and save

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@hellopablo
hellopablo merged commit 2b549ef into develop Sep 9, 2026
4 checks passed
@hellopablo
hellopablo deleted the feature/group-admin-tabs-develop branch September 9, 2026 20:00
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.

1 participant