From fd0e88dac095db4a962bf51493da81fe598f2ee6 Mon Sep 17 00:00:00 2001 From: Maria Grimaldi Date: Mon, 29 Jun 2026 12:04:16 +0200 Subject: [PATCH] feat: drop global roles from role filter The list endpoint no longer returns django-managed roles (openedx/openedx-authz#345), so these filter options have no effect. Remove Super Admin and Global Staff until proper staff/superuser listing is in place (openedx/openedx-authz#347). Co-Authored-By: Claude Sonnet 4.6 --- .../TableControlBar/TableControlBar.test.tsx | 6 +++--- src/authz-module/components/constants.ts | 13 ------------- src/authz-module/components/messages.ts | 5 ----- 3 files changed, 3 insertions(+), 21 deletions(-) diff --git a/src/authz-module/components/TableControlBar/TableControlBar.test.tsx b/src/authz-module/components/TableControlBar/TableControlBar.test.tsx index 4ca8b4cc..4d3a9268 100644 --- a/src/authz-module/components/TableControlBar/TableControlBar.test.tsx +++ b/src/authz-module/components/TableControlBar/TableControlBar.test.tsx @@ -90,9 +90,9 @@ describe('TableControlBar', () => { const rolesButton = screen.getByText('Select Roles'); expect(rolesButton).toBeInTheDocument(); await user.click(rolesButton); - const superAdminOption = screen.getByRole('checkbox', { name: /Super Admin/i }); - expect(superAdminOption).toBeInTheDocument(); - await user.click(superAdminOption); + const courseAdminOption = screen.getByRole('checkbox', { name: /Course Admin/i }); + expect(courseAdminOption).toBeInTheDocument(); + await user.click(courseAdminOption); expect(contextWithRolesFilter.columns[0].setFilter).toHaveBeenCalled(); }); diff --git a/src/authz-module/components/constants.ts b/src/authz-module/components/constants.ts index 21f43bf0..3218d163 100644 --- a/src/authz-module/components/constants.ts +++ b/src/authz-module/components/constants.ts @@ -3,19 +3,6 @@ import { Language, LibraryBooks, School } from '@openedx/paragon/icons'; import messages from './messages'; export const getRolesFiltersOptions = (intl: IntlShape) => [ - { - groupName: intl.formatMessage(messages['authz.team.members.table.group.global']), - groupIcon: Language, - displayName: 'Super Admin', - value: 'super_admin', - }, - { - groupName: intl.formatMessage(messages['authz.team.members.table.group.global']), - groupIcon: Language, - displayName: 'Global Staff', - value: 'global_staff', - }, - { groupName: intl.formatMessage(messages['authz.team.members.table.group.courses']), groupIcon: School, diff --git a/src/authz-module/components/messages.ts b/src/authz-module/components/messages.ts index 55f3b040..0bc9d62c 100644 --- a/src/authz-module/components/messages.ts +++ b/src/authz-module/components/messages.ts @@ -107,11 +107,6 @@ const messages = defineMessages({ defaultMessage: 'Libraries', description: 'Label for the "Libraries" group in the team members table filters', }, - 'authz.team.members.table.group.global': { - id: 'authz.team.members.table.group.global', - defaultMessage: 'Global', - description: 'Label for the "Global" group in the team members table filters', - }, 'authz.table.controlbar.filters.more.results': { id: 'authz.table.controlbar.filters.more.results', defaultMessage: 'Search to show more',