elan-ev / studip-opencast-plugin

This is an Stud.IP plugin for Opencast
https://elan-ev.github.io/studip-opencast-plugin/
GNU General Public License v2.0
9 stars 22 forks source link

Add warnings that other playlists will be affected when a video is marked for deletion or edited #885

Closed Sturek closed 7 months ago

Sturek commented 7 months ago

As videos exist only once and are linked to playlists, editing or deleting them may affect playlists on different courses. This may be unexpected for some users and it is easy to misremember which playlists will be affected. Thus, we need a message warning users that other playlists (in other courses) will be affected.

This warning should include:

This warning should only pop up when other playlists than the currently selected playlist would be affected. In the work place it should only pop up if the video has been added to any playlist.

Actions that should trigger this warning:

A similar warning should be triggered when the video is marked for deletion in the work place, if the video has been added to any playlists.

dennis531 commented 7 months ago

Actions that should trigger this warning:

kai-wng-uos commented 7 months ago

Instead of adding the warning to "Wiederherstellen", I would add it to "Zum Löschen markieren". Otherwise great!

dennis531 commented 7 months ago

In https://github.com/elan-ev/studip-opencast-plugin/pull/905#issue-2094079467 the changes are described in detail. The action "Zum Löschen markieren" contains a list of linked playlists at the bottom of the dialog. This list is only visible if the video is at least one playlist. The action "Wiederherstellen" works similar. It displays all playlists into which the video will be inserted after the restore.