From f5caf21d183434babd6d9e5c99146e33df856cc0 Mon Sep 17 00:00:00 2001
From: Aditya <aditya.gajbhiye.20@gmail.com>
Date: Wed, 27 Nov 2024 19:23:07 +0530
Subject: [PATCH 1/4] fix: showing doc prop btn when col desc

---
 .../documentationPropagation/DocumentationPropagation.tsx     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
index cd4960493..e8122d2ea 100644
--- a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
+++ b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
@@ -183,6 +183,10 @@ export const DocumentationPropagationButton = ({
     return null;
   }
 
+  if (!currColumnDescription) {
+    return null;
+  }
+
   return (
     <Drawer
       buttonProps={{ color: "primary", title: "Propagate documentation" }}

From 13cee5b1a6c28ebb43fe86abbc698f1bf6fd35e1 Mon Sep 17 00:00:00 2001
From: Aditya <aditya.gajbhiye.20@gmail.com>
Date: Wed, 27 Nov 2024 19:28:45 +0530
Subject: [PATCH 2/4] fix: styling

---
 .../DocumentationPropagation.tsx                    |  8 ++++----
 .../documentationPropagation/styles.module.scss     | 13 ++++++++++++-
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
index e8122d2ea..5d4fc8f72 100644
--- a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
+++ b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
@@ -197,16 +197,16 @@ export const DocumentationPropagationButton = ({
     >
       <Stack direction="column" className="gap-0 mb-2">
         <div className={styles.itemRow}>
-          <div className="fw-semibold">Model:</div>
+          <div>Model:</div>
           <div>{currentDocsData?.name}</div>
         </div>
         <div className={styles.itemRow}>
-          <div className="fw-semibold">Column:</div>
+          <div>Column:</div>
           <div>{name}</div>
         </div>
         {currColumnDescription && (
-          <div className={styles.itemRow}>
-            <div className="fw-semibold">Description:</div>
+          <div className={styles.colDesc}>
+            <div>Description:</div>
             <div>{currColumnDescription}</div>
           </div>
         )}
diff --git a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/styles.module.scss b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/styles.module.scss
index 084ec6460..2ff8c245a 100644
--- a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/styles.module.scss
+++ b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/styles.module.scss
@@ -9,7 +9,18 @@
   grid-template-columns: 1fr 3fr;
 
   :nth-child(1) {
-    font-weight: 500;
+    font-weight: 600;
+  }
+
+  :nth-child(2) {
+    line-break: anywhere;
+    color: var(--text-grey2-color);
+  }
+}
+
+.colDesc {
+  :nth-child(1) {
+    font-weight: 600;
   }
 
   :nth-child(2) {

From a1572ac650c2b4611dedddbfadf68419d63884f9 Mon Sep 17 00:00:00 2001
From: Aditya <aditya.gajbhiye.20@gmail.com>
Date: Wed, 27 Nov 2024 19:38:43 +0530
Subject: [PATCH 3/4] fix: no cll text

---
 .../DocumentationPropagation.tsx              | 131 ++++++++++--------
 1 file changed, 70 insertions(+), 61 deletions(-)

diff --git a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
index 5d4fc8f72..e477d81d0 100644
--- a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
+++ b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
@@ -211,67 +211,76 @@ export const DocumentationPropagationButton = ({
           </div>
         )}
       </Stack>
-      <Stack className="mb-2">
-        <Button color="primary" onClick={() => setAllColumnsValue(true)}>
-          Select All
-        </Button>
-        <Button color="primary" onClick={() => setAllColumnsValue(false)}>
-          Unselect All
-        </Button>
-      </Stack>
-      <Stack direction="column" className="gap-md">
-        {allColumns.map((item) => {
-          const key = item.model + "/" + item.column;
-          return (
-            <Stack key={key} className={styles.itemCard}>
-              <Input
-                type="checkbox"
-                checked={selectedColumns[key]}
-                onChange={() =>
-                  setSelectedColumns((prev) => ({
-                    ...prev,
-                    [key]: !prev[key],
-                  }))
-                }
-              />
-              <Stack direction="column" className="gap-0 w-100">
-                <div className={styles.itemRow}>
-                  <div>Model:</div>
-                  <div>{item.model.split(".").pop()}</div>
-                </div>
-                <div className={styles.itemRow}>
-                  <div>Column:</div>
-                  <div>{item.column}</div>
-                </div>
-                <div className={styles.itemRow}>
-                  <div>Description:</div>
-                  <div>{item.description}</div>
-                </div>
-              </Stack>
-            </Stack>
-          );
-        })}
-        {currColumns.length > 0 && (
-          <Button
-            color="primary"
-            outline
-            onClick={loadMoreDownstreamModels}
-            disabled={isLoading}
-          >
-            Load 3 more downstream levels
-          </Button>
-        )}
-        <Button
-          color="primary"
-          disabled={
-            Object.values(selectedColumns).filter((v) => Boolean(v)).length ===
-            0
-          }
-          onClick={() => propagateDocumentation()}
-        >
-          Propagate documentation to selected models
-        </Button>
-      </Stack>
+      {!isLoading && allColumns.length === 0 ? (
+        <div className="mt-4">
+          No downstream column level lineage detected to propagate the
+          documentation
+        </div>
+      ) : (
+        <>
+          <Stack className="mb-2">
+            <Button color="primary" onClick={() => setAllColumnsValue(true)}>
+              Select All
+            </Button>
+            <Button color="primary" onClick={() => setAllColumnsValue(false)}>
+              Unselect All
+            </Button>
+          </Stack>
+          <Stack direction="column" className="gap-md">
+            {allColumns.map((item) => {
+              const key = item.model + "/" + item.column;
+              return (
+                <Stack key={key} className={styles.itemCard}>
+                  <Input
+                    type="checkbox"
+                    checked={selectedColumns[key]}
+                    onChange={() =>
+                      setSelectedColumns((prev) => ({
+                        ...prev,
+                        [key]: !prev[key],
+                      }))
+                    }
+                  />
+                  <Stack direction="column" className="gap-0 w-100">
+                    <div className={styles.itemRow}>
+                      <div>Model:</div>
+                      <div>{item.model.split(".").pop()}</div>
+                    </div>
+                    <div className={styles.itemRow}>
+                      <div>Column:</div>
+                      <div>{item.column}</div>
+                    </div>
+                    <div className={styles.itemRow}>
+                      <div>Description:</div>
+                      <div>{item.description}</div>
+                    </div>
+                  </Stack>
+                </Stack>
+              );
+            })}
+            {currColumns.length > 0 && (
+              <Button
+                color="primary"
+                outline
+                onClick={loadMoreDownstreamModels}
+                disabled={isLoading}
+              >
+                Load 3 more downstream levels
+              </Button>
+            )}
+            <Button
+              color="primary"
+              disabled={
+                Object.values(selectedColumns).filter((v) => Boolean(v))
+                  .length === 0
+              }
+              onClick={() => propagateDocumentation()}
+            >
+              Propagate documentation to selected models
+            </Button>
+          </Stack>
+        </>
+      )}
     </Drawer>
   );
 };

From d8ce15079f86536dea186806901043ce7ff77e8c Mon Sep 17 00:00:00 2001
From: Aditya <aditya.gajbhiye.20@gmail.com>
Date: Wed, 27 Nov 2024 21:27:23 +0530
Subject: [PATCH 4/4] feat: added saved sucessfully message

---
 .../documentationPropagation/DocumentationPropagation.tsx     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
index e477d81d0..0300cba38 100644
--- a/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
+++ b/webview_panels/src/modules/documentationEditor/components/documentationPropagation/DocumentationPropagation.tsx
@@ -60,6 +60,7 @@ export const DocumentationPropagationButton = ({
   const [allColumns, setAllColumns] = useState<DocsItem[]>([]);
   const [currColumns, setCurrColumns] = useState<DocsItem[]>([]);
   const [isLoading, setIsLoading] = useState(false);
+  const [isSaved, setIsSaved] = useState(false);
   const [tableMetadata, setTableMetadata] = useState<TableMetadata[]>([]);
   const [testsMetadata, setTestsMetadata] = useState<Record<string, unknown>>(
     {},
@@ -72,6 +73,7 @@ export const DocumentationPropagationButton = ({
     setAllColumns([]);
     setCurrColumns(startColumn);
     setTableMetadata([]);
+    setIsSaved(false);
   }, [currentDocsData?.uniqueId, name]);
 
   const loadMoreDownstreamModels = async () => {
@@ -165,6 +167,7 @@ export const DocumentationPropagationButton = ({
     }
 
     await executeRequestInSync("saveDocumentationBulk", { models: req });
+    setIsSaved(true);
   };
 
   const setAllColumnsValue = (value: boolean) => {
@@ -278,6 +281,7 @@ export const DocumentationPropagationButton = ({
             >
               Propagate documentation to selected models
             </Button>
+            {isSaved && <div>Saved documentation successfully</div>}
           </Stack>
         </>
       )}