fix(minor): update skip icon for executions (#14809)

open-metadata · Jan 23, 2024 · c11fb01 · c11fb01
1 parent 9c92bea
commit c11fb01
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/openmetadata-ui/src/main/resources/ui/src/assets/svg/skipped-badge.svg b/openmetadata-ui/src/main/resources/ui/src/assets/svg/skipped-badge.svg
diff --git a/openmetadata-ui/src/main/resources/ui/src/utils/PipelineDetailsUtils.ts b/openmetadata-ui/src/main/resources/ui/src/utils/PipelineDetailsUtils.ts
@@ -37,7 +37,7 @@ export const getStatusBadgeIcon = (status?: StatusType) => {
       return Icons.FAIL_BADGE;
 
     case StatusType.Pending:
-      return Icons.PENDING_BADGE;
+      return Icons.SKIPPED_BADGE;
 
     case StatusType.Skipped:
       return Icons.SKIPPED_BADGE;

diff --git a/openmetadata-ui/src/main/resources/ui/src/utils/executionUtils.tsx b/openmetadata-ui/src/main/resources/ui/src/utils/executionUtils.tsx
@@ -13,7 +13,7 @@
 
 import { Col, Row, Space, Tooltip } from 'antd';
 import { DataNode } from 'antd/lib/tree';
-import { groupBy, isUndefined, map, toLower, uniqueId } from 'lodash';
+import { groupBy, isUndefined, map, toLower } from 'lodash';
 import React, { ReactNode } from 'react';
 import { MenuOptions } from '../constants/execution.constants';
 import {
@@ -188,12 +188,12 @@ export const getTreeData = (
   const viewElements = map(viewData, (value, key) => ({
     key,
     value: (
-      <Row gutter={16} key={uniqueId()}>
+      <Row gutter={16} key={key}>
         <Col>
           <div className="execution-node-container">
             {value.map((status) => (
               <Tooltip
-                key={uniqueId()}
+                key={`${status.timestamp}-${status.executionStatus}`}
                 placement="top"
                 title={
                   <Space direction="vertical">