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.

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

Add Opencast Playlists #955

Merged
merged 20 commits into from May 10, 2024
Merged

Add Opencast Playlists #955

merged 20 commits into from May 10, 2024

Conversation

dennis531
Copy link
Collaborator

@dennis531 dennis531 commented Mar 25, 2024

This draft pull request shows a prototype for the support of Opencast playlists. There are still some open todos and disscussion points open.

TODOs:

  • Request and refresh current playlist from Opencast in backend not in browser as students have no write access to playlists
  • Perform write actions in backend to improve error handling
  • Migration flag for playlists. Not needed, as only playlists that are assigned to an Opencast instance are considered in the cronjob (config_id)
  • Update oc api endpoints in migration
  • Cronjob: Ignore new playlists in Opencast
  • Remove playlist video if write permission on playlist is granted. Already implemented this way.
  • Add video playlist if user has write permission on playlist and video. Already implemented this way.
  • Should the playlist to Opencast migration be outsourced in a separate script? We'll leave it as it is for now and test it.
  • Should a video appear more than once in a playlist?

@dennis531 dennis531 self-assigned this Mar 25, 2024
@dennis531 dennis531 added v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:feature labels Mar 25, 2024
@dennis531 dennis531 marked this pull request as ready for review April 10, 2024 15:20
@dennis531
Copy link
Collaborator Author

The Pull-Request for the playlists api in the opencast-php-library is still open. Thus the required version 1.6 in composer.json is not available yet.

@dennis531 dennis531 requested a review from tgloeggl April 10, 2024 15:28
Copy link

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

@tgloeggl tgloeggl merged commit 9c73929 into elan-ev:master May 10, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:conflicts type:feature v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants