Merge pull request #1895 from automatisch/fix-appkey-error-in-flowrow

fix: remove unnecessary appKey in FlowRow and FlowContextMenu
This commit is contained in:
Ali BARIN
2024-05-31 12:54:13 +02:00
committed by GitHub
2 changed files with 18 additions and 8 deletions

View File

@@ -28,9 +28,12 @@ function ContextMenu(props) {
variables: { input: { id: flowId } }, variables: { input: { id: flowId } },
}); });
if (appKey) {
await queryClient.invalidateQueries({ await queryClient.invalidateQueries({
queryKey: ['apps', appKey, 'flows'], queryKey: ['apps', appKey, 'flows'],
}); });
}
enqueueSnackbar(formatMessage('flow.successfullyDuplicated'), { enqueueSnackbar(formatMessage('flow.successfullyDuplicated'), {
variant: 'success', variant: 'success',
SnackbarProps: { SnackbarProps: {
@@ -56,9 +59,12 @@ function ContextMenu(props) {
}, },
}); });
if (appKey) {
await queryClient.invalidateQueries({ await queryClient.invalidateQueries({
queryKey: ['apps', appKey, 'flows'], queryKey: ['apps', appKey, 'flows'],
}); });
}
enqueueSnackbar(formatMessage('flow.successfullyDeleted'), { enqueueSnackbar(formatMessage('flow.successfullyDeleted'), {
variant: 'success', variant: 'success',
}); });
@@ -110,7 +116,7 @@ ContextMenu.propTypes = {
]).isRequired, ]).isRequired,
onDeleteFlow: PropTypes.func, onDeleteFlow: PropTypes.func,
onDuplicateFlow: PropTypes.func, onDuplicateFlow: PropTypes.func,
appKey: PropTypes.string.isRequired, appKey: PropTypes.string,
}; };
export default ContextMenu; export default ContextMenu;

View File

@@ -38,20 +38,24 @@ function FlowRow(props) {
const contextButtonRef = React.useRef(null); const contextButtonRef = React.useRef(null);
const [anchorEl, setAnchorEl] = React.useState(null); const [anchorEl, setAnchorEl] = React.useState(null);
const { flow, onDuplicateFlow, onDeleteFlow, appKey } = props; const { flow, onDuplicateFlow, onDeleteFlow, appKey } = props;
const handleClose = () => { const handleClose = () => {
setAnchorEl(null); setAnchorEl(null);
}; };
const onContextMenuClick = (event) => { const onContextMenuClick = (event) => {
event.preventDefault(); event.preventDefault();
event.stopPropagation(); event.stopPropagation();
event.nativeEvent.stopImmediatePropagation(); event.nativeEvent.stopImmediatePropagation();
setAnchorEl(contextButtonRef.current); setAnchorEl(contextButtonRef.current);
}; };
const createdAt = DateTime.fromMillis(parseInt(flow.createdAt, 10)); const createdAt = DateTime.fromMillis(parseInt(flow.createdAt, 10));
const updatedAt = DateTime.fromMillis(parseInt(flow.updatedAt, 10)); const updatedAt = DateTime.fromMillis(parseInt(flow.updatedAt, 10));
const isUpdated = updatedAt > createdAt; const isUpdated = updatedAt > createdAt;
const relativeCreatedAt = createdAt.toRelative(); const relativeCreatedAt = createdAt.toRelative();
const relativeUpdatedAt = updatedAt.toRelative(); const relativeUpdatedAt = updatedAt.toRelative();
return ( return (
<> <>
<Card sx={{ mb: 1 }} data-test="flow-row"> <Card sx={{ mb: 1 }} data-test="flow-row">
@@ -127,7 +131,7 @@ FlowRow.propTypes = {
flow: FlowPropType.isRequired, flow: FlowPropType.isRequired,
onDeleteFlow: PropTypes.func, onDeleteFlow: PropTypes.func,
onDuplicateFlow: PropTypes.func, onDuplicateFlow: PropTypes.func,
appKey: PropTypes.string.isRequired, appKey: PropTypes.string,
}; };
export default FlowRow; export default FlowRow;