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

Set publication variables in analyze-MP #5419

Merged
merged 2 commits into from Nov 22, 2023

Conversation

KatrinIhler
Copy link
Member

@KatrinIhler KatrinIhler commented Nov 8, 2023

Similar to #5243 except this creates variables that contain which publication channels are part of the media package. These new variables are not created by default.

The use case for this was that I wanted to publish everything if the event wasn't published before, including encoding and all, but only merge specific assets into the publication if it existed. Here's a snippet if you want to test this for yourself:

    <operation
      id="include"
      description="Publish to Engage"
      if="NOT ${publication_engage_player_exists}">
      <configurations>
        <configuration key="workflow-id">partial-publish</configuration>
      </configurations>
    </operation>

    <operation
      id="include"
      description="Update Engage publication"
      if="${publication_engage_player_exists}">
      <configurations>
        <configuration key="workflow-id">partial-publish-generated-captions</configuration>
      </configurations>
    </operation>

Since the other one went into 14.x I'm aiming this at that branch as well, but I can aim at develop instead if somebody disagrees.

This also contains a small change to the docs that explains which WOHs can generate workflow variables, because I keep forgetting this.

@KatrinIhler KatrinIhler added enhancement ELAN Pull requests originating from ELAN e.V. java Pull requests that update Java code labels Nov 8, 2023
@Arnei Arnei self-assigned this Nov 22, 2023
Copy link
Member

@Arnei Arnei left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me and works.

@Arnei Arnei merged commit 9c2611b into opencast:develop Nov 22, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ELAN Pull requests originating from ELAN e.V. enhancement java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants