Document or remove undocumented workflow operation handlers

Description

From: Lars Kiesow lkiesow@uos.de
Sent: 13 November 2017 12:47 AM
To: dev@opencast.org
Subject: Re: [OC Dev] [#proposal] Drop Undocumented Workflow Handler

Hi everyone,
this proposal has passed by now.

As promised, here is the list of operations currently missing an entry
in the documented list of workflow operation handlers:

https://docs.opencast.org/develop/admin/workflowoperationhandlers/

A few of these entries are quite surprising and I'm pretty confident
some will get documentation.

  • capture (obsolete)

  • coverart

  • distribute-acl PR #1802 (drops WOH distribute-acl)

  • embed

  • error-resolution [PR #1801] (added documentation)

  • failing PR #1804 (added documentation)

  • include PR #1805 (added documentation)

  • ingest (obsolete)

  • log

  • publish-youtube

  • retract-youtube

  • searchindex-remove

  • send-email [PR #1793] (documentation was there, now linked correctly)

  • timelinepreviews [PR #1791] (documentation was there, now linked correctly)

  • watermark

Remember that if you want to keep any of these, you have to provide
documentation until the end of the year. I'll reach out to our non-devs
by creating a proper announcement about this in a minute.

Best regards,
Lars

Activity

Show:
Stephen Marquard
December 4, 2017, 11:00 AM

In review of PR #1765 for MH-11980, it appears that capture and ingest workflows are obsolete (amongst other things, they refer to html UI elements which are removed in #1765). It looks like matterhorn-capture-workflowoperation should be removed entirely (the whole module), and matterhorn-ingest-workflowoperation/src/main/java/org/opencastproject/workflow/handler/ingest/IngestWorkflowOperationHandler.java

https://bitbucket.org/opencast-community/opencast/pull-requests/1765/

Rute Santos
November 16, 2017, 9:50 PM

error-resolution: PR #1801

Stephen Marquard
November 13, 2017, 9:14 AM

To find WOHs:

  1. grep -r "property name=\"workflow.operation\"" modules//src/ | awk '{print $4}' | sed s/value=// | sed s/[\"\>\/]//g | sort

Fixed and reviewed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Lars Kiesow

Reporter

Stephen Marquard