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

Instant save in PlaylistAddToCourse and VideoLinkToPlaylists #838

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

ssrahn
Copy link
Collaborator

@ssrahn ssrahn commented Nov 13, 2023

#790

Wie in #790 (comment) erwähnt, gibt es noch Dialoge bei denen das Abspeichern direkt bei der Auswahl geschehen sollte.

  • Wurde jetzt Umgesetzt für die Komponenten VideoLinkToPlaylists und PlaylistAddToCourseDialog
  • Die remove-Funktionen konnten zu fehlerhaften Verhalten führen, wenn der Aufruf noch nicht verarbeitet wurde, währenddessen aber neue Elemente hinzugefügt werden - habe sichergestellt, dass das nicht mehr passieren kann
  • Der VideoAddToSeminar Dialog wird momentan nicht genutzt - hab diesen gelöscht

@ssrahn ssrahn added type:bug v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:usability labels Nov 13, 2023
@ssrahn ssrahn requested a review from tgloeggl November 13, 2023 14:57
@ssrahn ssrahn self-assigned this Nov 13, 2023
Copy link

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

@tgloeggl tgloeggl merged commit 5c7d157 into master Nov 27, 2023
3 checks passed
return ApiService.put('playlists/' + params.token + '/courses', {courses: params.courses})
},

async removePlaylistFromCourse(context, params) {
Copy link
Member

Choose a reason for hiding this comment

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

Gibt es einen bestimmten Grund, dass diese Methode entfernt wurde?

@ssrahn ssrahn deleted the fix790-2 branch December 11, 2023 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug type:usability 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