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

Playlists Changes #848

Merged
merged 10 commits into from
Dec 11, 2023
Merged

Playlists Changes #848

merged 10 commits into from
Dec 11, 2023

Conversation

dennis531
Copy link
Collaborator

Changes:

  • New add playlist dialog in course
    • Shows playlists of work place
  • Paging for playlists table
  • Filter + Search for playlists
    • Available filters: Tags, Courses, Lecturers

Fixes:

Playlists add dialog in course:
image
image

Dennis Benz added 8 commits December 4, 2023 10:07
@dennis531 dennis531 requested a review from tgloeggl December 4, 2023 09:26
@dennis531 dennis531 self-assigned this Dec 4, 2023
@dennis531 dennis531 added type:bug v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:feature labels Dec 4, 2023
@dennis531 dennis531 changed the title Changes for playlists Playlists Changes Dec 4, 2023
Copy link
Member

@tgloeggl tgloeggl left a comment

Choose a reason for hiding this comment

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

Erstmal nur rein visuell gechecked, Funktionscheck folgt.

lib/Routes/Course/CourseAddPlaylist.php Outdated Show resolved Hide resolved
vueapp/components/Playlists/PlaylistAddNewCard.vue Outdated Show resolved Hide resolved
vueapp/components/Playlists/PlaylistsTable.vue Outdated Show resolved Hide resolved
@dennis531
Copy link
Collaborator Author

Danke für die Änderungsvorschläge!

@tgloeggl tgloeggl merged commit 45b0aac into elan-ev:master Dec 11, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug 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.

2 participants