Unsupervised/unsupervised-scheduler
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases 12 Wiki Activity

Group the studio admin sidebar into three sections #26

Merged
thatguygriff merged 1 commits from feature/admin-menu-separator into main 2026-06-08 14:13:21 +00:00
Conversation 0 Commits 1 Files Changed 1
+67 -8
thatguygriff commented 2026-06-08 14:07:35 +00:00
Owner
Copy Link
Copy Source

Cosmetic admin-sidebar change: set the studio menus apart from the core WordPress items and group them into three sections, using wp-menu-separator dividers (the same mechanism core uses before Appearance).

Result

──────────  (separator, pos 29)
Studio Settings · Policies · Invites · Offerings   (30–33)
──────────  (separator, pos 34)
Students · Group Classes · Payments                (35–37)
──────────  (separator, pos 38)
Scheduler · My Availability · My Lessons           (39–41)
  • Studio menus reordered to the requested order; the instructor menus (My Availability, My Lessons) follow Scheduler.
  • Each separator is only added when the current user can see a menu in the following section, to avoid orphaned dividers for instructors/students.
  • The $menu write carries a scoped phpcs:ignore (writing to $menu is the supported way to add an admin separator).

Verification

composer test (148 tests), composer cs, PHPStan level 6, and no-debug all pass.

🤖 Generated with Claude Code

Cosmetic admin-sidebar change: set the studio menus apart from the core WordPress items and group them into three sections, using `wp-menu-separator` dividers (the same mechanism core uses before Appearance). ## Result ``` ────────── (separator, pos 29) Studio Settings · Policies · Invites · Offerings (30–33) ────────── (separator, pos 34) Students · Group Classes · Payments (35–37) ────────── (separator, pos 38) Scheduler · My Availability · My Lessons (39–41) ``` - Studio menus reordered to the requested order; the instructor menus (My Availability, My Lessons) follow Scheduler. - Each separator is only added when the current user can see a menu in the following section, to avoid orphaned dividers for instructors/students. - The `$menu` write carries a scoped `phpcs:ignore` (writing to `$menu` is the supported way to add an admin separator). ## Verification `composer test` (148 tests), `composer cs`, PHPStan level 6, and no-debug all pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
thatguygriff added 1 commit 2026-06-08 14:07:36 +00:00
Group the studio admin menu into three sections with separators
CI / No Debug Code (pull_request) Successful in 3s
Details
CI / Tests (PHP 8.1) (pull_request) Successful in 46s
Details
CI / Coding Standards (pull_request) Successful in 54s
Details
CI / Tests (PHP 8.3) (pull_request) Successful in 51s
Details
CI / Tests (PHP 8.2) (pull_request) Successful in 59s
Details
CI / PHPStan (pull_request) Successful in 1m14s
Details
CI / Build Plugin Zip (pull_request) Has been skipped
Details
ca458bb8ae 
Add sidebar separators (wp-menu-separator) that set the studio menus apart
from the core WordPress items and split them into three sections:

  [sep] Studio Settings · Policies · Invites · Offerings
  [sep] Students · Group Classes · Payments
  [sep] Scheduler  (then the instructor menus: My Availability, My Lessons)

Separators live at positions 29 / 34 / 38; each is only added when the user
can see a menu in the following section, to avoid orphaned dividers.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
thatguygriff merged commit d229b83b23 into main 2026-06-08 14:13:21 +00:00
thatguygriff deleted branch feature/admin-menu-separator 2026-06-08 14:13:21 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 enhancement
Extends an existing feature
 feature
New user-facing capability
 payments
Billing, Stripe, receipts, reporting
 security
Security vulnerability or hardening
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
thatguygriff (James Griffin-Allwood)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Unsupervised/unsupervised-scheduler#26