Skip to content

Do not filter selected role from ACL dropdown#1499

Merged
gregorydlogan merged 1 commit into
opencast:r/19.xfrom
Arnei:acl-dropdown-usability
Jan 23, 2026
Merged

Do not filter selected role from ACL dropdown#1499
gregorydlogan merged 1 commit into
opencast:r/19.xfrom
Arnei:acl-dropdown-usability

Conversation

@Arnei

@Arnei Arnei commented Jan 9, 2026

Copy link
Copy Markdown
Member

In the ACL tab in e.g. the event details, if you open the dropdown for a selected role like ROLE_USER_ADMIN, the selected role does not appear in the dropdown. This in confusing, as our dropdowns do not behave in this manner.
This patch makes it so that the role does appear in the dropdown.

Bildschirmfoto vom 2026-01-09 11-39-36

How to test this

Can be tested as is. Check that the acl dropdowns still work.

In the ACL tab in e.g. the event details, if you open the
dropdown for a selected role
like ROLE_USER_ADMIN,
the selected role does not appear in the dropdown.
This in confusing, as our dropdowns do not behave
in this manner.
This patch makes it so that the role *does* appear in the
dropdown.
@Arnei Arnei added the type:usability Improves the UX label Jan 9, 2026
@github-actions

github-actions Bot commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-1499

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org ghcr.io/opencast/opencast-admin-interface:pr-1499

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

@github-actions

github-actions Bot commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

This pull request is deployed at test.admin-interface.opencast.org/1499/2026-01-09_10-41-41/ .
It might take a few minutes for it to become available.

@gregorydlogan gregorydlogan self-assigned this Jan 23, 2026
@gregorydlogan gregorydlogan merged commit a5b031d into opencast:r/19.x Jan 23, 2026
9 checks passed
jduehring pushed a commit to jduehring/opencast-admin-interface that referenced this pull request Apr 16, 2026
…ce into r/19.x

Pull request opencast#1499

  Do not filter selected role from ACL dropdown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:usability Improves the UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants