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

Playlist Copy #893

Merged
merged 12 commits into from
Jan 18, 2024
Merged

Playlist Copy #893

merged 12 commits into from
Jan 18, 2024

Conversation

dennis531
Copy link
Collaborator

@dennis531 dennis531 commented Jan 16, 2024 •
edited
Loading

Implements the concept in #886 (comment).

Changes:

  • Hide linking of playlist in "Wiedergabeliste hinzufügen" dialog
  • Hide playlists in contents sidebar
  • Add copy action in "Wiedergabeliste hinzufügen" dialog
  • "Videos/Wiedergabelisten übertragen" action: Copy instead of link playlists to other courses
  • Show name and semester of first course in playlist tables and in "Verknüpfungen" video dialog. The other courses will be ignored in the playlist table.

Fixes:

  • Currently, a course member only sees the default playlist or a playlist with at least read permission. This check is removed so that all playlist are visible for course members.

Migration:

  • Duplicate playlists with more than one linked course to ensure that playlists belong to only one course
  • Copy playlists with tags, videos and perms

Screenshots:
image
image
image

Closes #886

Dennis Benz added 12 commits January 15, 2024 14:40
Add api route for copy playlist, ref elan-ev#886
179d664
Hide linking action in playlist add dialog, ref elan-ev#886
793be96
Add playlist copy action in playlist add dialog, ref elan-ev#886
0f70b59
Copy playlists instead of linking when transferring all playlists of …
1056a4c 
…a course, ref elan-ev#886
Prevent duplicate tags in tag copy
db5037a
Show all course playlists for course members
a94bad1 
Currently, a course member only sees the default playlist or a playlist with at least read permission
Hide playlists view in contents,ref elan-ev#886
9b5ff9f
Copy video order to playlist copy, ref elan-ev#886
8807ee5
Add migration for duplicating playlists, ref elan-ev#886
01590f6 
Duplicate playlists with more than one linked course to ensure that playlists belong to only one course
Show name and semester of first course in playlist table, ref elan-ev…
9825b41 
…#886

Assume that a playlist has only one course
Fix name of PlaylistsCopyCard
ad2696e
Show name and semester of first course in video link to playlist dial…
a2daa70 
…og, ref elan-ev#886

Assume that a playlist has only one course
@dennis531 dennis531 self-assigned this Jan 16, 2024
@dennis531 dennis531 added type:enhancement v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x labels Jan 16, 2024
@dennis531 dennis531 changed the title Playlist copy Playlist Copy Jan 16, 2024
This was referenced Jan 16, 2024
Merged
Closed
@tgloeggl tgloeggl merged commit 0c43bcf into elan-ev:master Jan 18, 2024
3 of 4 checks passed
This was referenced Jan 19, 2024
Closed
Closed
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

Assignees

@dennis531 dennis531

Labels
type:enhancement v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Default action for adding existing playlists to other courses should be copying - remove linking feature
2 participants
@dennis531 @tgloeggl