allow not redirecting to home page at all in interstitial component and do not error out if next task does not exist

spiffworkflow-backend/src/spiffworkflow_backend/routes/tasks_controller.py

@@ -430,20 +430,21 @@ def _interstitial_stream(process_instance: ProcessInstanceModel) -> Generator[st
             break  # No more tasks to report
 
     spiff_task = processor.next_task()
-    task = ProcessInstanceService.spiff_task_to_api_task(processor, processor.next_task())
-    if task.id not in reported_ids:
-        try:
-            instructions = render_instructions(spiff_task)
-        except Exception as e:
-            api_error = ApiError(
-                error_code="engine_steps_error",
-                message=f"Failed to complete an automated task. Error was: {str(e)}",
-                status_code=400,
-            )
-            yield render_data("error", api_error)
-            raise e
-        task.properties = {"instructionsForEndUser": instructions}
-        yield render_data("task", task)
+    if spiff_task is not None:
+        task = ProcessInstanceService.spiff_task_to_api_task(processor, spiff_task)
+        if task.id not in reported_ids:
+            try:
+                instructions = render_instructions(spiff_task)
+            except Exception as e:
+                api_error = ApiError(
+                    error_code="engine_steps_error",
+                    message=f"Failed to complete an automated task. Error was: {str(e)}",
+                    status_code=400,
+                )
+                yield render_data("error", api_error)
+                raise e
+            task.properties = {"instructionsForEndUser": instructions}
+            yield render_data("task", task)
 
 
 def get_ready_engine_step_count(bpmn_process_instance: BpmnWorkflow) -> int:

spiffworkflow-backend/src/spiffworkflow_backend/services/process_instance_service.py

@@ -514,7 +514,6 @@ class ProcessInstanceService:
         add_docs_and_forms: bool = False,
         calling_subprocess_task_id: str | None = None,
     ) -> Task:
-        """Spiff_task_to_api_task."""
         task_type = spiff_task.task_spec.description
 
         props = {}

spiffworkflow-frontend/src/components/ProcessInterstitial.tsx

@@ -15,12 +15,14 @@ type OwnProps = {
   processInstanceId: number;
   processInstanceShowPageUrl: string;
   allowRedirect: boolean;
+  redirectToHomeWhenNoTasks?: boolean;
 };
 
 export default function ProcessInterstitial({
   processInstanceId,
   allowRedirect,
   processInstanceShowPageUrl,
+  redirectToHomeWhenNoTasks = true,
 }: OwnProps) {
   const [data, setData] = useState<any[]>([]);
   const [lastTask, setLastTask] = useState<any>(null);
@@ -49,7 +51,6 @@ export default function ProcessInterstitial({
         }
       },
       onclose() {
-        console.log('The state is closed.');
         setState('CLOSED');
       },
     });
@@ -161,7 +162,7 @@ export default function ProcessInterstitial({
 
   /** In the event there is no task information and the connection closed,
    * redirect to the home page. */
-  if (state === 'CLOSED' && lastTask === null) {
+  if (state === 'CLOSED' && lastTask === null && redirectToHomeWhenNoTasks) {
     navigate(`/tasks`);
   }
 

spiffworkflow-frontend/src/routes/ProcessInstanceShow.tsx

@@ -1113,6 +1113,7 @@ export default function ProcessInstanceShow({ variant }: OwnProps) {
         <ProcessInterstitial
           processInstanceId={processInstance.id}
           processInstanceShowPageUrl={processInstanceShowPageBaseUrl}
+          redirectToHomeWhenNoTasks={false}
           allowRedirect={false}
         />
         <br />