From f94801ab45530eb933a00cbfe41f57819bb7cff4 Mon Sep 17 00:00:00 2001
From: David Leek <david@getunleash.io>
Date: Tue, 12 Aug 2025 09:16:07 +0200
Subject: [PATCH] fix: move update approvers after cr change state (#10490)

---
 .../EnvironmentChangeRequest.tsx                       | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/frontend/src/component/changeRequest/ChangeRequestSidebar/EnvironmentChangeRequest/EnvironmentChangeRequest.tsx b/frontend/src/component/changeRequest/ChangeRequestSidebar/EnvironmentChangeRequest/EnvironmentChangeRequest.tsx
index d57f103a38..31c25b7970 100644
--- a/frontend/src/component/changeRequest/ChangeRequestSidebar/EnvironmentChangeRequest/EnvironmentChangeRequest.tsx
+++ b/frontend/src/component/changeRequest/ChangeRequestSidebar/EnvironmentChangeRequest/EnvironmentChangeRequest.tsx
@@ -82,6 +82,11 @@ export const EnvironmentChangeRequest: FC<{
     const sendToReview = async (project: string) => {
         setDisabled(true);
         try {
+            await changeState(project, environmentChangeRequest.id, 'Draft', {
+                state: 'In review',
+                comment: commentText,
+            });
+
             if (reviewers && reviewers.length > 0) {
                 await updateRequestedApprovers(
                     project,
@@ -89,11 +94,6 @@ export const EnvironmentChangeRequest: FC<{
                     reviewers.map((reviewer) => reviewer.id),
                 );
             }
-
-            await changeState(project, environmentChangeRequest.id, 'Draft', {
-                state: 'In review',
-                comment: commentText,
-            });
         } catch (e) {
             setDisabled(false);
         }