Skip to content

Conversation

@andrey-canon
Copy link
Collaborator

Description

This update includes the following changes:

  1. Make the sysadmin panel tenant-aware (courses are now displayed based on the course_org_filter value).
  2. Allow access to the sysadmin panel based on the user's course access role.

Migration pr of #3

Issue # 1296

How to test

  1. Set the course_org_filter with the right organization
  2. Set the course access role for a non admin user
    image
  3. Go to /sysadmin and check the courses sections with and admin user a non admin user

Before

image

After

image

Copy link
Collaborator

@johanseto johanseto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Working in my local env

org_aware_filter

Screencast.from.02-12-25.09.28.18.webm

course_access_role

Screencast.from.02-12-25.16.31.52.webm

@johanseto
Copy link
Collaborator

@andrey-canon andrey-canon merged commit a6c3d8d into open-release/teak.nelp Dec 2, 2025
1 check failed
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp December 2, 2025 22:45 — with GitHub Actions Inactive
@andrey-canon andrey-canon deployed to open-release/teak.nelp December 2, 2025 22:45 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants