Skip to content

Add metadata change detection to optimize video update process #1363

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ferishili
Copy link
Contributor

This PR fixes #1350,

To keep things simple, we first create a non-reactive copy of the event object. We then compare this copy with the current event to detect whether any metadata-related fields have changed. Based on this comparison, we pass a flag to the backend, which decides whether the update metadata call needs to be triggered.

To keep things simple, we first create a non-reactive copy of the event object. We then compare this copy with the current event to detect whether any metadata-related fields have changed. Based on this comparison, we pass a flag to the backend, which decides whether the update metadata call needs to be triggered.
@ferishili ferishili requested a review from tgloeggl May 8, 2025 12:14
@ferishili ferishili self-assigned this May 8, 2025
@ferishili ferishili added type:enhancement v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x labels May 8, 2025
@ferishili
Copy link
Contributor Author

ferishili commented Jun 4, 2025

Discussion:

  • Do everything in the backend instead of having the flag from frontend.
  • rely on the local copy of the event data, so avoid another call to OC!
    [UPDATE]: changes have been applied in 9df7910

@ferishili
Copy link
Contributor Author

[UPDATE]
The point in dicussion are now applied, therefore, @tgloeggl please proceed with the review!
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement 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.

Changing video visibility triggers republish-metadata workflow
1 participant