Cutting and Review are skipped when config is set to do so

Steps to reproduce

Steps to reproduce:
1. Create event with default workflow
2. Leave Cutting and Review config options checked

Actual Results:
The Cutting operation is skipped

Expected Results:

The workflow pauses for Cutting and Review

Workaround (if any):

in ng-partial-publish the editor operation has the config option "interactive" set to "false"

Activity

Show:
Tobias Wunden
June 12, 2015, 5:24 PM

Hi James,

I am not sure I completely understand. The workflow panel has two options (cut and review) that basically mark the recording as "needs cutting", "needs review". Whether the recording is published before that or is holding, is subject to the "Publish right away" setting. So I am not sure if we are actually looking at a bug here? (other than a documentation bug, maybe)?

Thanks,
Tobias

James Perrin
June 15, 2015, 9:47 AM

Ok, so I see that the Cutting and Review options do nothing other than add comments that the Video needs cutting and review.

However these options are checked by default and if the recording needs needs Cutting and Review then surely it should not be Published before there has been and opportunity to actual cut and review it! In my mind selecting Cutting and/or Review implies Hold.

Lars Kiesow
June 15, 2015, 2:49 PM

From what I understood it correctly, the idea of a hold state has completely been removed in 2.0. Even when I use the old admin ui in 2.0 sending something to the video editor, the hold state is just skipped and the workflow runs through without waiting for user input.

James Perrin
June 15, 2015, 2:59 PM

Again Ok, but selecting Cutting and Review still implies that files shouldn't be published until Cutting and Review!

Lars Kiesow
June 15, 2015, 5:33 PM

I huess that is correct. A better wording could be “Add review notification”, “Add edit notification”? That would make it clearer that only a comment is added that the event should be reviewed and “hold” needs to be selected if it should not automatically be published.

Assignee

James Perrin

Reporter

James Perrin

Severity

Incorrectly Functioning With Workaround

Tags (folksonomy)

None

Components

Fix versions

Affects versions

Priority

Critical
Configure