From 3fd735ac7e96ef1680859f4d63d1c65c8be95314 Mon Sep 17 00:00:00 2001
From: Mateusz Kwasniewski <kwasniewski.mateusz@gmail.com>
Date: Tue, 23 Jan 2024 09:32:48 +0100
Subject: [PATCH] fix: move useEffect before renders (#5995)

---
 .../NewFeatureStrategyForm.tsx                | 36 +++++++++----------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/frontend/src/component/feature/FeatureStrategy/FeatureStrategyForm/NewFeatureStrategyForm.tsx b/frontend/src/component/feature/FeatureStrategy/FeatureStrategyForm/NewFeatureStrategyForm.tsx
index b36595aebb..d590f30ee2 100644
--- a/frontend/src/component/feature/FeatureStrategy/FeatureStrategyForm/NewFeatureStrategyForm.tsx
+++ b/frontend/src/component/feature/FeatureStrategy/FeatureStrategyForm/NewFeatureStrategyForm.tsx
@@ -232,6 +232,24 @@ export const NewFeatureStrategyForm = ({
         });
     });
 
+    const stickiness =
+        strategy?.parameters && 'stickiness' in strategy?.parameters
+            ? String(strategy.parameters.stickiness)
+            : 'default';
+
+    useEffect(() => {
+        setStrategy((prev) => ({
+            ...prev,
+            variants: (strategy.variants || []).map((variant) => ({
+                stickiness,
+                name: variant.name,
+                weight: variant.weight,
+                payload: variant.payload,
+                weightType: variant.weightType,
+            })),
+        }));
+    }, [stickiness, JSON.stringify(strategy.variants)]);
+
     const foundEnvironment = feature.environments.find(
         (environment) => environment.name === environmentId,
     );
@@ -354,24 +372,6 @@ export const NewFeatureStrategyForm = ({
     const showVariants =
         strategy.parameters && 'stickiness' in strategy.parameters;
 
-    const stickiness =
-        strategy?.parameters && 'stickiness' in strategy?.parameters
-            ? String(strategy.parameters.stickiness)
-            : 'default';
-
-    useEffect(() => {
-        setStrategy((prev) => ({
-            ...prev,
-            variants: (strategy.variants || []).map((variant) => ({
-                stickiness,
-                name: variant.name,
-                weight: variant.weight,
-                payload: variant.payload,
-                weightType: variant.weightType,
-            })),
-        }));
-    }, [stickiness, JSON.stringify(strategy.variants)]);
-
     return (
         <>
             <StyledHeaderBox>