1
0
mirror of https://github.com/Unleash/unleash.git synced 2025-08-23 13:46:45 +02:00

Add default to strategy change

This commit is contained in:
Thomas Heartman 2025-07-03 12:57:15 +02:00
parent 88bdb41eaa
commit 46e8a174ae
3 changed files with 38 additions and 8 deletions

View File

@ -96,7 +96,15 @@ export const FeatureChange: FC<{
change: IFeatureChange; change: IFeatureChange;
feature: IChangeRequestFeature; feature: IChangeRequestFeature;
onNavigate?: () => void; onNavigate?: () => void;
}> = ({ index, change, feature, changeRequest, actions, onNavigate }) => { }> = ({
index,
change,
feature,
changeRequest,
actions,
onNavigate,
isDefaultChange,
}) => {
const lastIndex = feature.defaultChange const lastIndex = feature.defaultChange
? feature.changes.length + 1 ? feature.changes.length + 1
: feature.changes.length; : feature.changes.length;
@ -190,7 +198,7 @@ export const FeatureChange: FC<{
)} )}
{change.action === 'updateEnabled' && ( {change.action === 'updateEnabled' && (
<StatusChangeComponent <StatusChangeComponent
isDefaultChange isDefaultChange={isDefaultChange}
enabled={change.payload.enabled} enabled={change.payload.enabled}
actions={actions} actions={actions}
/> />
@ -204,6 +212,7 @@ export const FeatureChange: FC<{
change.action === 'updateStrategy' ? ( change.action === 'updateStrategy' ? (
<StrategyChangeComponent <StrategyChangeComponent
actions={actions} actions={actions}
isDefaultChange={isDefaultChange}
change={change} change={change}
featureName={feature.name} featureName={feature.name}
environmentName={changeRequest.environment} environmentName={changeRequest.environment}

View File

@ -262,7 +262,8 @@ const UpdateStrategy: FC<{
const AddStrategy: FC<{ const AddStrategy: FC<{
change: IChangeRequestAddStrategy; change: IChangeRequestAddStrategy;
actions?: ReactNode; actions?: ReactNode;
}> = ({ change, actions }) => ( isDefaultChange?: boolean;
}> = ({ change, actions, isDefaultChange }) => (
<> <>
<ChangeItemCreateEditDeleteWrapper> <ChangeItemCreateEditDeleteWrapper>
<ChangeItemInfo> <ChangeItemInfo>
@ -288,7 +289,14 @@ const AddStrategy: FC<{
/> />
</div> </div>
</ChangeItemInfo> </ChangeItemInfo>
<div>{actions}</div> <div>
{isDefaultChange ? (
<Typography variant='body2' color='text.secondary'>
Default strategy will be added
</Typography>
) : null}
{actions}
</div>
</ChangeItemCreateEditDeleteWrapper> </ChangeItemCreateEditDeleteWrapper>
<StrategyExecution strategy={change.payload} /> <StrategyExecution strategy={change.payload} />
{change.payload.variants?.length ? ( {change.payload.variants?.length ? (
@ -316,6 +324,7 @@ export const LegacyStrategyChange: FC<{
featureName: string; featureName: string;
projectId: string; projectId: string;
changeRequestState: ChangeRequestState; changeRequestState: ChangeRequestState;
isDefaultChange?: boolean;
}> = ({ }> = ({
actions, actions,
change, change,
@ -323,6 +332,7 @@ export const LegacyStrategyChange: FC<{
environmentName, environmentName,
projectId, projectId,
changeRequestState, changeRequestState,
isDefaultChange,
}) => { }) => {
const currentStrategy = useCurrentStrategy( const currentStrategy = useCurrentStrategy(
change, change,
@ -334,7 +344,11 @@ export const LegacyStrategyChange: FC<{
return ( return (
<> <>
{change.action === 'addStrategy' && ( {change.action === 'addStrategy' && (
<AddStrategy change={change} actions={actions} /> <AddStrategy
change={change}
actions={actions}
isDefaultChange={isDefaultChange}
/>
)} )}
{change.action === 'deleteStrategy' && ( {change.action === 'deleteStrategy' && (
<DeleteStrategy <DeleteStrategy

View File

@ -239,13 +239,14 @@ const UpdateStrategy: FC<{
const AddStrategy: FC<{ const AddStrategy: FC<{
change: IChangeRequestAddStrategy; change: IChangeRequestAddStrategy;
isDefaultChange?: boolean;
actions?: ReactNode; actions?: ReactNode;
}> = ({ change, actions }) => ( }> = ({ change, isDefaultChange, actions }) => (
<> <>
<ChangeItemWrapper> <ChangeItemWrapper>
<ChangeItemInfo> <ChangeItemInfo>
<AddedStrategy disabled={change.payload?.disabled}> <AddedStrategy disabled={change.payload?.disabled}>
Adding strategy Adding {isDefaultChange && 'default'} strategy
</AddedStrategy> </AddedStrategy>
<ChangeStrategyName <ChangeStrategyName
name={change.payload.name} name={change.payload.name}
@ -300,6 +301,7 @@ export const StrategyChange: FC<{
featureName: string; featureName: string;
projectId: string; projectId: string;
changeRequestState: ChangeRequestState; changeRequestState: ChangeRequestState;
isDefaultChange?: boolean;
}> = ({ }> = ({
actions, actions,
change, change,
@ -307,6 +309,7 @@ export const StrategyChange: FC<{
environmentName, environmentName,
projectId, projectId,
changeRequestState, changeRequestState,
isDefaultChange,
}) => { }) => {
const currentStrategy = useCurrentStrategy( const currentStrategy = useCurrentStrategy(
change, change,
@ -328,7 +331,11 @@ export const StrategyChange: FC<{
return ( return (
<StyledTabs> <StyledTabs>
{change.action === 'addStrategy' && ( {change.action === 'addStrategy' && (
<AddStrategy change={change} actions={actionsWithTabs} /> <AddStrategy
change={change}
actions={actionsWithTabs}
isDefaultChange={isDefaultChange}
/>
)} )}
{change.action === 'deleteStrategy' && ( {change.action === 'deleteStrategy' && (
<DeleteStrategy <DeleteStrategy