Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2024-12-22 19:07:54 +01:00
Code Issues Projects Releases Wiki Activity

hide discard button (#2506)

Browse Source
This commit is contained in:
Mateusz Kwasniewski 2022-11-23 12:16:20 +01:00 committed by GitHub
parent f87feb9b84
commit 98a9a770d2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 76 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
frontend/src/component/changeRequest/ChangeRequest/ChangeRequest.tsx
View File

@ -8,6 +8,7 @@ import { formatUnknownError } from 'utils/formatUnknownError';
import useToast from 'hooks/useToast';
import type { IChangeRequest } from '../changeRequest.types';
import {
Discard,
StrategyAddedChange,
StrategyDeletedChange,
StrategyEditedChange,
@ -18,6 +19,7 @@ import {
} from 'utils/strategyNames';
import { hasNameField } from '../changeRequest.types';
import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender';
import { useChangeRequestOpen } from '../../../hooks/api/getters/useChangeRequestOpen/useChangeRequestOpen';
interface IChangeRequestProps {
changeRequest: IChangeRequest;
@ -91,6 +93,10 @@ export const ChangeRequest: VFC<IChangeRequestProps> = ({
}
};
const showDiscard =
!['Cancelled', 'Applied'].includes(changeRequest.state) &&
changeRequest.features.flatMap(feature => feature.changes).length > 1;
return (
<Box>
{changeRequest.features?.map(featureToggleChange => (
@ -131,12 +137,34 @@ export const ChangeRequest: VFC<IChangeRequestProps> = ({
{change.action === 'updateEnabled' && (
<ToggleStatusChange
enabled={change.payload.enabled}
onDiscard={onDiscard(change.id)}
discard={
<ConditionallyRender
condition={showDiscard}
show={
<Discard
onDiscard={onDiscard(
change.id
)}
/>
}
/>
}
/>
)}
{change.action === 'addStrategy' && (
<StrategyAddedChange
onDiscard={onDiscard(change.id)}
discard={
<ConditionallyRender
condition={showDiscard}
show={
<Discard
onDiscard={onDiscard(
change.id
)}
/>
}
/>
}
>
<GetFeatureStrategyIcon
strategyName={change.payload.name}
@ -149,7 +177,18 @@ export const ChangeRequest: VFC<IChangeRequestProps> = ({
)}
{change.action === 'deleteStrategy' && (
<StrategyDeletedChange
onDiscard={onDiscard(change.id)}
discard={
<ConditionallyRender
condition={showDiscard}
show={
<Discard
onDiscard={onDiscard(
change.id
)}
/>
}
/>
}
>
{hasNameField(change.payload) && (
<>
@ -167,7 +206,18 @@ export const ChangeRequest: VFC<IChangeRequestProps> = ({
)}
{change.action === 'updateStrategy' && (
<StrategyEditedChange
onDiscard={onDiscard(change.id)}
discard={
<ConditionallyRender
condition={showDiscard}
show={
<Discard
onDiscard={onDiscard(
change.id
)}
/>
}
/>
}
>
<GetFeatureStrategyIcon
strategyName={change.payload.name}

22
frontend/src/component/changeRequest/ChangeRequestOverview/ChangeRequestFeatureToggleChange/StrategyChange.tsx
View File

@ -1,5 +1,5 @@
import { Box, Link, styled, Typography } from '@mui/material';
import { FC } from 'react';
import { FC, ReactNode } from 'react';
interface IStrategyChangeProps {
onDiscard: () => void;
@ -16,15 +16,15 @@ const ChangeItemInfo: FC = styled(Box)(({ theme }) => ({
gap: theme.spacing(1),
}));
const Discard: FC<IStrategyChangeProps> = ({ onDiscard }) => (
export const Discard: FC<IStrategyChangeProps> = ({ onDiscard }) => (
<Box>
<Link onClick={onDiscard}>Discard</Link>
</Box>
);
export const StrategyAddedChange: FC<IStrategyChangeProps> = ({
export const StrategyAddedChange: FC<{ discard?: ReactNode }> = ({
children,
onDiscard,
discard,
}) => {
return (
<ChangeItemWrapper>
@ -36,14 +36,14 @@ export const StrategyAddedChange: FC<IStrategyChangeProps> = ({
</Typography>
{children}
</ChangeItemInfo>
<Discard onDiscard={onDiscard} />
{discard}
</ChangeItemWrapper>
);
};
export const StrategyEditedChange: FC<IStrategyChangeProps> = ({
export const StrategyEditedChange: FC<{ discard?: ReactNode }> = ({
children,
onDiscard,
discard,
}) => {
return (
<ChangeItemWrapper>
@ -51,13 +51,13 @@ export const StrategyEditedChange: FC<IStrategyChangeProps> = ({
<Typography>Editing strategy:</Typography>
{children}
</ChangeItemInfo>
<Discard onDiscard={onDiscard} />
{discard}
</ChangeItemWrapper>
);
};
export const StrategyDeletedChange: FC<IStrategyChangeProps> = ({
onDiscard,
export const StrategyDeletedChange: FC<{ discard?: ReactNode }> = ({
discard,
children,
}) => {
return (
@ -68,7 +68,7 @@ export const StrategyDeletedChange: FC<IStrategyChangeProps> = ({
</Typography>
{children}
</ChangeItemInfo>
<Discard onDiscard={onDiscard} />
{discard}
</ChangeItemWrapper>
);
};

22
frontend/src/component/changeRequest/ChangeRequestOverview/ChangeRequestFeatureToggleChange/ToggleStatusChange.tsx
View File

@ -1,17 +1,16 @@
import { VFC } from 'react';
import { Link, Box } from '@mui/material';
import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender';
import { ReactNode, VFC } from 'react';
import { Box } from '@mui/material';
import { Badge } from 'component/common/Badge/Badge';
import { ChangeItemWrapper } from './StrategyChange';
interface IPlaygroundResultsTable {
interface IToggleStatusChange {
enabled: boolean;
onDiscard?: () => void;
discard?: ReactNode;
}
export const ToggleStatusChange: VFC<IPlaygroundResultsTable> = ({
export const ToggleStatusChange: VFC<IToggleStatusChange> = ({
enabled,
onDiscard,
discard,
}) => {
return (
<ChangeItemWrapper>
@ -21,14 +20,7 @@ export const ToggleStatusChange: VFC<IPlaygroundResultsTable> = ({
{enabled ? ' Enabled' : 'Disabled'}
</Badge>
</Box>
<ConditionallyRender
condition={Boolean(onDiscard)}
show={
<Box>
<Link onClick={onDiscard}>Discard</Link>
</Box>
}
/>
{discard}
</ChangeItemWrapper>
);
};

5
frontend/src/component/changeRequest/ChangeRequestOverview/ChangeRequestOverview.tsx
View File

@ -133,7 +133,10 @@ export const ChangeRequestOverview: FC = () => {
<StyledPaper elevation={0}>
<StyledInnerContainer>
Changes
<ChangeRequest changeRequest={changeRequest} />
<ChangeRequest
changeRequest={changeRequest}
onRefetch={refetchChangeRequest}
/>
{changeRequest.comments?.map(comment => (
<ChangeRequestComment
key={comment.id}