Skip to content

Task - implement course team & groups #323

Description

@MaferMazu
No description provided.

Activity

  1. changed the title [-]Task - implement course updates[/-] [+]Task - implement course teams[/+] on Jun 2, 2026
  2. moved this to In Grooming in RBAC AuthZ Boardon Jun 2, 2026
  3. dcoa commented on Aug 7, 2026

    @dcoa
    Contributor

    @BryanttV @gviedma-aulasneo besides redirecting the user to Admin Console ( that we covered here openedx/frontend-app-authoring#3151 (comment)) from Authoring what other changes should be part of this scope?

    I can review if there is any other place in Authoring that needs redirect but I want to make sure the scope.

  4. BryanttV commented on Aug 10, 2026

    @BryanttV
    Contributor

    Hi @dcoa, we have to validate 3 permissions in this issue:

    Category Permission Course Editor Course Auditor
    Course Team & Groups courses.view_course_team ✔ ✔
    courses.manage_group_configurations ✔ ✗
    courses.manage_course_team ✗ ✗

    Is validation working correctly throughout the entire interface — including menus, URL access, etc. — in both the Admin Console and Authoring? If so, we can close this issue.

  5. changed the title [-]Task - implement course teams[/-] [+]Task - implement course team & groups[/+] on Aug 10, 2026
  6. dcoa commented on Aug 11, 2026

    @dcoa
    Contributor

    Here we need also a view permission for group_configuration or do you think we should use the global course view permission? @BryanttV @gviedma-aulasneo

    Image

    The Course Team workflow is already covered we only need to update Group Configuration

    Course configuration refinement

    1. ContentGroupsSection/index.jsx: Accept canManageGroupConfigurations prop, pass readOnly to ContentGroupCard, and hide the "Add new group" button + hide EmptyPlaceholder (or pass props to it) when no permission.
    2. EmptyPlaceholder/index.jsx: Accept a canManage prop to change message and hide the button.
    3. empty-placeholder/messages.ts: Add the new message "No group configurations have been added to this course yet."
    4. Tests update.

    Note

    This does not include the decision of view status, once a decision is made about we can include the Page level access visibility, and navigation links validations.

  7. dcoa commented on Aug 12, 2026

    @dcoa
    Contributor

    @BryanttV just an update when I was checking the refinement also noticed that the group configurations API respond 403 for Course Auditor role.

  8. BryanttV commented on Aug 12, 2026

    @BryanttV
    Contributor

    @dcoa, yes. Since the Course Auditor doesn't have the courses.manage_group_configurations permission, the REST API returns a 403. We need to add the new read-only permission to the role. I'm tracking it in this issue: #387

  9. moved this from In Grooming to Blocked in RBAC AuthZ Boardon Aug 13, 2026
  10. moved this from Blocked to In Progress in RBAC AuthZ Boardon Aug 27, 2026
  11. moved this from In Progress to Ready for Review in RBAC AuthZ Boardon Sep 1, 2026
  12. moved this from Ready for Review to Done in RBAC AuthZ Boardon Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions