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

Show course videos in cw #1132

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Show course videos in cw #1132

wants to merge 4 commits into from

Conversation

dennis531
Copy link
Collaborator

Changes:

  • In Courseware block, initially filter the videos of the course if a course exists
  • In workplace, admins should see only the videos of their courses and videos with explicit permissions
    • Fixed Bug: Admins could see all videos of the system due to a missing video join
  • In cw block and workplace, show tutors and lecturers their videos from their courses and videos with explicit permissions

Screenshot of cw block filter:
image

@dennis531
Show initially only course videos in courseware block
12f3964
@dennis531
Fix video filter for admins
589d035 
Currently, admins could see all videos due to a missing join with the video table.
@dennis531
Show tutors, lecturers and admins their course videos and videos with…
699ede0 
… perms in cw block and workplace
@dennis531 dennis531 added type:bug v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:usability labels Jan 20, 2025
tgloeggl
tgloeggl requested changes Jan 27, 2025
View reviewed changes
Copy link
Member

@tgloeggl tgloeggl left a comment •
edited
Loading

Choose a reason for hiding this comment

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

  • It should be possible to re-add the filter if it is removed.

@dennis531
Copy link
Collaborator Author

dennis531 commented Jan 29, 2025 •
edited
Loading

Now it is possible to re-add the filter:

Screencast.from.2025-01-29.09-17-38.webm

@dennis531 dennis531 requested a review from tgloeggl January 29, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgloeggl tgloeggl Awaiting requested review from tgloeggl

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
type:bug type:usability v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
SOP V3 Priorisierung
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dennis531 @tgloeggl