Skip to content

Remove READ column from ACL modal UI #1368

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

priya-tik
Copy link
Contributor

@priya-tik priya-tik commented Jul 10, 2025

This PR removes the READ column from the Access Control List (ACL) modal user interface.
image

Copy link
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-1368

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-1368

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.

Copy link
Contributor

This pull request is deployed at test.admin-interface.opencast.org/1368/2025-07-10_13-15-39/ .
It might take a few minutes for it to become available.

Copy link
Contributor

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

@gregorydlogan gregorydlogan added the type:usability Improves the UX label Jul 15, 2025
@gregorydlogan
Copy link
Member

Discussion at the technical meeting suggested that this reads like the user can't do anything in the ACL. I'm inclined to agree. @Arnei suggested in the technical meeting that this was possible with config changes alone though, so perhaps this becomes something where we just change hte default?

@Arnei
Copy link
Member

Arnei commented Jul 16, 2025

To be precise, I did not say you can configure the removal of the READ column. I said that you can configure the read/write columns to be read-only. If read-only, the respective columns are still visible, but the checkboxes are greyed out and uninteractable.

Config file: https://github.com/opencast/opencast/blob/develop/etc/listproviders/acl.default.create.properties
The READ column is read-only per default, you can see this behaviour on e.g. develop.opencast.org.

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.

3 participants