The rendering of node status in a dynamic workflow is not functioning…

… correctly (#723) fix: dynamic nodes status Signed-off-by: James <[email protected]> Signed-off-by: 4nalog <[email protected]>
flyteorg · Mar 30, 2023 · 45c6386 · 45c6386
1 parent 0fab0d7
commit 45c6386
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/packages/console/src/components/Executions/ExecutionDetails/ExecutionNodeViews.tsx b/packages/console/src/components/Executions/ExecutionDetails/ExecutionNodeViews.tsx
@@ -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';
@@ -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 = () => {