From 317a8f17112d6eafcef2a76b603926abf29d67e6 Mon Sep 17 00:00:00 2001
From: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>
Date: Wed, 30 Nov 2022 23:14:31 +0530
Subject: [PATCH] Fix #8993 Add Datainsight ingestion option available even
 after a pipeline already running (#9075)

* Fix #8993 Add Datainsight ingestion option available even after a pipeline already running

* Remove tailwind class

* Remove tailwind

* refactor : the code to check if Data InSight Ingestion Exists
---
 .../src/components/Ingestion/Ingestion.component.tsx | 12 ++++++++++++
 .../ui/src/components/dropdown/DropDownList.tsx      |  5 +++--
 .../src/main/resources/ui/src/styles/spacing.less    | 11 +++++++++++
 3 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Ingestion/Ingestion.component.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Ingestion/Ingestion.component.tsx
index 5c2b2f267b9..e2521969834 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/Ingestion/Ingestion.component.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/Ingestion/Ingestion.component.tsx
@@ -252,6 +252,14 @@ const Ingestion: React.FC<IngestionProps> = ({
     }
   };
 
+  const isDataSightIngestionExists = useMemo(
+    () =>
+      ingestionData.some(
+        (ingestion) => ingestion.pipelineType === PipelineType.DataInsight
+      ),
+    [ingestionData]
+  );
+
   const getAddIngestionButton = (type: PipelineType) => {
     return (
       <Button
@@ -297,6 +305,10 @@ const Ingestion: React.FC<IngestionProps> = ({
           <DropDownList
             horzPosRight
             dropDownList={types.map((type) => ({
+              disabled:
+                type === PipelineType.DataInsight
+                  ? isDataSightIngestionExists
+                  : false,
               name: `${t('label.add')} ${startCase(type)} ${
                 type === PipelineType.ElasticSearchReindex
                   ? ''
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx
index 55de1f7d5e6..751cd1cdb3c 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/dropdown/DropDownList.tsx
@@ -137,10 +137,11 @@ const DropDownList: FunctionComponent<DropDownListProp> = ({
       <div
         aria-disabled={item.disabled as boolean}
         className={classNames(
-          'tw-text-gray-700 tw-flex tw-px-4 tw-py-2 tw-text-sm hover:tw-bg-body-hover tw-cursor-pointer',
+          'text-body flex px-4 py-2 text-sm hover:tw-bg-body-hover',
           !isNil(value) && item.value === value ? 'tw-bg-primary-lite' : null,
           {
-            'tw-cursor-not-allowed': item.disabled,
+            'opacity-60 cursor-not-allowed': item.disabled,
+            'cursor-pointer': !item.disabled,
           }
         )}
         data-testid="list-item"
diff --git a/openmetadata-ui/src/main/resources/ui/src/styles/spacing.less b/openmetadata-ui/src/main/resources/ui/src/styles/spacing.less
index 0e6ca27adb2..a260366ae8c 100644
--- a/openmetadata-ui/src/main/resources/ui/src/styles/spacing.less
+++ b/openmetadata-ui/src/main/resources/ui/src/styles/spacing.less
@@ -417,6 +417,17 @@
 .p-b-lg {
   padding-bottom: @padding-lg;
 }
+
+.px-4 {
+  padding-left: @padding-md;
+  padding-right: @padding-md;
+}
+
+.py-2 {
+  padding-top: @padding-xs;
+  padding-bottom: @padding-xs;
+}
+
 .take-to-right {
   position: absolute;
   right: 15px;