Set manager_bypass to False, remove guard-role for Manager for mark-as-done, validate and refuse transitions

src/collective/task/profiles/default/workflows/basic_task_workflow/definition.xml

@@ -38,7 +38,6 @@
   <action url="" category="workflow" icon="" i18n:translate="">Mark as done</action>
   <guard>
     <guard-role>Editor</guard-role>
-    <guard-role>Manager</guard-role>
   </guard>
  </transition>
 </dc-workflow>

src/collective/task/profiles/default/workflows/task_workflow/definition.xml

@@ -4,7 +4,7 @@
              description="Task workflow"
              state_variable="review_state"
              initial_state="todo"
-             manager_bypass="True"
+             manager_bypass="False"
              xmlns:i18n="http://xml.zope.org/namespaces/i18n"
              i18n:domain="plone">
  <permission>Modify portal content</permission>

src/collective/task/profiles/default/workflows/validation_workflow/definition.xml

@@ -50,14 +50,12 @@
   <action url="" category="workflow" icon="" i18n:translate="">Validate</action>
   <guard>
     <guard-role>Editor</guard-role>
-    <guard-role>Manager</guard-role>
   </guard>
  </transition>
  <transition transition_id="refuse" title="Refuse" new_state="refused" trigger="USER" before_script="" after_script="">
   <action url="" category="workflow" icon="" i18n:translate="">Refuse</action>
   <guard>
     <guard-role>Editor</guard-role>
-    <guard-role>Manager</guard-role>
   </guard>
  </transition>
 </dc-workflow>