mirror of
				https://github.com/Unleash/unleash.git
				synced 2025-10-27 11:02:16 +01:00 
			
		
		
		
	fix: move useEffect before renders (#5995)
This commit is contained in:
		
							parent
							
								
									5305c868e5
								
							
						
					
					
						commit
						3fd735ac7e
					
				| @ -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> | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user