From 733159b8ec6e515a16066cb6c7fecaf54a2a8636 Mon Sep 17 00:00:00 2001 From: Mateusz Kwasniewski Date: Thu, 27 Jul 2023 09:12:30 +0200 Subject: [PATCH] feat: strategy variants infop (#4348) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## About the changes Strategy variants info and link to docs. Added short info about what variants allow to do, how strategy variants relate to the feature variants, that modern SDK is required and link to docs. Screenshot 2023-07-26 at 14 51 06 ### Important files ## Discussion points --- .../StrategyTypes/StrategyVariants.tsx | 29 +++++++++++++++++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/frontend/src/component/feature/StrategyTypes/StrategyVariants.tsx b/frontend/src/component/feature/StrategyTypes/StrategyVariants.tsx index d6e6ad2e54..c89d0450a8 100644 --- a/frontend/src/component/feature/StrategyTypes/StrategyVariants.tsx +++ b/frontend/src/component/feature/StrategyTypes/StrategyVariants.tsx @@ -7,10 +7,11 @@ import { UPDATE_FEATURE_ENVIRONMENT_VARIANTS } from '../../providers/AccessProvi import { v4 as uuidv4 } from 'uuid'; import { WeightType } from '../../../constants/variantTypes'; import { useRequiredPathParam } from 'hooks/useRequiredPathParam'; -import { styled, Typography, useTheme } from '@mui/material'; +import { styled, Typography, useTheme, Link } from '@mui/material'; import { useRequiredQueryParam } from 'hooks/useRequiredQueryParam'; import { IFeatureStrategy } from 'interfaces/strategy'; import SplitPreviewSlider from './SplitPreviewSlider/SplitPreviewSlider'; +import { HelpIcon } from '../../common/HelpIcon/HelpIcon'; const StyledVariantForms = styled('div')({ display: 'flex', @@ -86,8 +87,32 @@ export const StrategyVariants: FC<{ return ( <> - + Variants + + + Variants allow to attach one or more values to + this strategy. Variants at the strategy level + override variants at the feature level. Make + sure to use recent SDK that supports strategy + variants.{' '} + + + Learn more + + + } + /> {variantsEdit.map((variant, i) => (