Skip to content

Commit

Permalink
The rendering of node status in a dynamic workflow is not functioning…
Browse files Browse the repository at this point in the history
… correctly (#723)

fix: dynamic nodes status

Signed-off-by: James <[email protected]>
  • Loading branch information
james-union authored Mar 15, 2023
1 parent a8ee3b2 commit 80edd63
Showing 1 changed file with 7 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { DataError } from 'components/Errors/DataError';
import { useTabState } from 'components/hooks/useTabState';
import { secondaryBackgroundColor } from 'components/Theme/constants';
import { Execution } from 'models/Execution/types';
import { keyBy } from 'lodash';
import { clone, keyBy, merge } from 'lodash';
import { LargeLoadingSpinner } from 'components/common/LoadingSpinner';
import { FilterOperation } from 'models/AdminEntity/types';
import { NodeExecutionDetailsContextProvider } from '../contextProvider/NodeExecutionDetails';
Expand Down Expand Up @@ -77,7 +77,12 @@ export const ExecutionNodeViews: React.FC<ExecutionNodeViewsProps> = ({
nodeExecutionsQuery.data,
'scopedId',
);
setCurrentNodeExecutionsById(currentNodeExecutionsById);
const prevNodeExecutionsById = clone(nodeExecutionsById);
const newNodeExecutionsById = merge(
prevNodeExecutionsById,
currentNodeExecutionsById,
);
setCurrentNodeExecutionsById(newNodeExecutionsById);
}, [nodeExecutionsQuery.data]);

const LoadingComponent = () => {
Expand Down

0 comments on commit 80edd63

Please sign in to comment.