Fix engagement status sync to consider specific steps before statuses

EarlyConversations -> FinalizingCompletion should change the engagement to FinalizingCompletion,
even though the status just became Active.

Author: CarsonF

src/components/engagement/handlers/update-engagement-status.handler.ts

@@ -14,19 +14,10 @@ import { EngagementRepository } from '../engagement.repository';
 import { EngagementService } from '../engagement.service';
 
 const changes: Change[] = [
-  {
-    to: { status: ProjectStatus.Active },
-    newStatus: EngagementStatus.Active,
-  },
   {
     to: { step: ProjectStep.ActiveChangedPlan },
     newStatus: EngagementStatus.ActiveChangedPlan,
   },
-  {
-    from: { status: ProjectStatus.Active },
-    to: { status: ProjectStatus.InDevelopment },
-    newStatus: EngagementStatus.InDevelopment,
-  },
   {
     to: { step: ProjectStep.DiscussingChangeToPlan },
     newStatus: EngagementStatus.DiscussingChangeToPlan,
@@ -63,6 +54,15 @@ const changes: Change[] = [
     to: { step: ProjectStep.Terminated },
     newStatus: EngagementStatus.Terminated,
   },
+  {
+    to: { status: ProjectStatus.Active },
+    newStatus: EngagementStatus.Active,
+  },
+  {
+    from: { status: ProjectStatus.Active },
+    to: { status: ProjectStatus.InDevelopment },
+    newStatus: EngagementStatus.InDevelopment,
+  },
 ];
 
 type Change = RequireAtLeastOne<{ from: Condition; to: Condition }> & {