Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update video ACLs in OC when adding to new course #1083

Closed
dennis531 opened this issue Nov 25, 2024 · 1 comment · Fixed by #1178
Closed

Update video ACLs in OC when adding to new course #1083

dennis531 opened this issue Nov 25, 2024 · 1 comment · Fixed by #1178
Assignees
@tgloeggl
Labels
type:feature v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Milestone
V3.23

Comments

@dennis531
Copy link
Collaborator

dennis531 commented Nov 25, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.
Adding a video to a playlist of a new course doesn't trigger the course ACLs to be updated on the Opencast server. There is currently no call to Opencast. It results that other user will not be able to open the video in the player.

Steps to reproduce:

  1. Upload video in Arbeitsplatz
  2. Wait until video is processed
  3. Link this video into a coure
  4. Change to another course user
  5. Open video with this user
  6. See error:

Image

Describe the solution you'd like
A possible solution could be to not show the video to users who don't have explicit permission on this video until the new Acls are updated on the OC server. In addition, the update process need to be triggered in such cases.
@tgloeggl tgloeggl self-assigned this Dec 9, 2024
@dennis531 dennis531 mentioned this issue Dec 11, 2024
Merged
@tgloeggl tgloeggl added this to the V3.23 milestone Dec 13, 2024
@tgloeggl tgloeggl added the v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x label Dec 13, 2024
tgloeggl added a commit that referenced this issue Jan 27, 2025
@tgloeggl
fixes #1083
f5aac6c
@dennis531 dennis531 changed the title Update video ACLs when adding to new course Update video ACLs in OC when adding to new course Jan 28, 2025
@dennis531
Copy link
Collaborator Author

Still does not work. I updated the description as the old one was very unclear.

@dennis531 dennis531 reopened this Jan 28, 2025
@dennis531 dennis531 moved this from Testing to Todo in SOP V3 Priorisierung Jan 28, 2025
@tgloeggl tgloeggl linked a pull request Feb 3, 2025 that will close this issue
Update video acls in oc when adding to new course #1178
Merged
@tgloeggl tgloeggl mentioned this issue Feb 3, 2025
Merged
@github-project-automation github-project-automation bot moved this from Todo to Testing in SOP V3 Priorisierung Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tgloeggl tgloeggl

Labels
type:feature v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
SOP V3 Priorisierung
Status: Testing
Milestone
V3.23
Development

Successfully merging a pull request may close this issue.

Update video acls in oc when adding to new course elan-ev/studip-opencast-plugin
2 participants
@tgloeggl @dennis531