multiple course group managers

[simonLeary42]

No description provided.

[Copilot]

Pull request overview

Updates the unity-course worker script to support assigning multiple PI group managers when creating a new course user and its PI group.

Changes:

• Accepts a comma-separated list of manager UIDs instead of a single UID.
• Creates/validates multiple manager UnityUser objects and attempts to add each to the PI group as a manager.

Suppressed comments (1)

workers/unity-course.php:71

• This block uses invalid PHP for (... in ...) syntax, and it also references $manager (undefined after switching to $managers) and $manager_uid (not defined in this loop) when setting mail / adding managers. Select a primary manager for the single-valued mail attribute and iterate managers with foreach ($managers as $manager_uid => $manager) when adding managers.

for ($managers as $manager) {
    $course_pi_group->newUserRequest($manager, false);
    $course_pi_group->approveUser($manager);
    $course_pi_group->addManagerUID($manager_uid);   
}

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.