Missing WorkflowPassCompleted enum element in WorkflowPersistenceBeha…

…vior.cs (#4480) * Update WorkflowPersistenceBehavior.cs * Update PersistWorkflow.cs
elsa-workflows · Oct 7, 2023 · e0a4cd9 · e0a4cd9
1 parent 0fbeebe
commit e0a4cd9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/core/Elsa.Abstractions/Models/WorkflowPersistenceBehavior.cs b/src/core/Elsa.Abstractions/Models/WorkflowPersistenceBehavior.cs
@@ -1,4 +1,4 @@
-using Newtonsoft.Json;
+using Newtonsoft.Json;
 using Newtonsoft.Json.Converters;
 
 namespace Elsa.Models
@@ -15,7 +15,12 @@ public enum WorkflowPersistenceBehavior
         /// Workflow instances are persisted after the workflow completed a burst of execution.
         /// </summary>
         WorkflowBurst,
-
+
+        /// <summary>
+        /// Workflow instances are persisted after the workflow executed scheduled activities.
+        /// </summary>
+        WorkflowPassCompleted,
+
         /// <summary>
         /// Workflow instances are persisted after each activity that executed.
         /// </summary>

diff --git a/src/core/Elsa.Core/Handlers/PersistWorkflow.cs b/src/core/Elsa.Core/Handlers/PersistWorkflow.cs
@@ -27,7 +27,7 @@ public async Task Handle(WorkflowExecutionPassCompleted notification, Cancellati
         {
             var behavior = notification.WorkflowExecutionContext.WorkflowBlueprint.PersistenceBehavior;
 
-            if (behavior == WorkflowPersistenceBehavior.ActivityExecuted || notification.ActivityExecutionContext.ActivityBlueprint.PersistWorkflow)
+            if (behavior == WorkflowPersistenceBehavior.ActivityExecuted || behavior == WorkflowPersistenceBehavior.WorkflowPassCompleted || notification.ActivityExecutionContext.ActivityBlueprint.PersistWorkflow)
                 await SaveWorkflowAsync(notification.WorkflowExecutionContext.WorkflowInstance, cancellationToken);
         }