From 9a249e887dddc10daf7c7f367bb945a0b59936e2 Mon Sep 17 00:00:00 2001
From: Muhammed Mustafa <mustafa@nocodb.com>
Date: Mon, 23 Oct 2023 08:49:26 +0000
Subject: [PATCH 1/2] fix: Hide col for project erd view and show in table erd
 view

---
 packages/nc-gui/components/dashboard/settings/DataSources.vue | 2 +-
 packages/nc-gui/components/dashboard/settings/Erd.vue         | 3 ++-
 packages/nc-gui/components/dlg/ProjectErd.vue                 | 2 +-
 packages/nc-gui/components/smartsheet/details/Erd.vue         | 2 +-
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/packages/nc-gui/components/dashboard/settings/DataSources.vue b/packages/nc-gui/components/dashboard/settings/DataSources.vue
index d646149590..1f2f76cbcf 100644
--- a/packages/nc-gui/components/dashboard/settings/DataSources.vue
+++ b/packages/nc-gui/components/dashboard/settings/DataSources.vue
@@ -554,7 +554,7 @@ const isEditBaseModalOpen = computed({
       />
       <GeneralModal v-model:visible="isErdModalOpen" size="large">
         <div class="h-[80vh]">
-          <LazyDashboardSettingsErd :source-id="activeBaseId" />
+          <LazyDashboardSettingsErd :source-id="activeBaseId" :show-all-columns="false" />
         </div>
       </GeneralModal>
       <GeneralModal v-model:visible="isMetaDataModal" size="medium">
diff --git a/packages/nc-gui/components/dashboard/settings/Erd.vue b/packages/nc-gui/components/dashboard/settings/Erd.vue
index 547a9a138f..3a7cef91fb 100644
--- a/packages/nc-gui/components/dashboard/settings/Erd.vue
+++ b/packages/nc-gui/components/dashboard/settings/Erd.vue
@@ -1,11 +1,12 @@
 <script setup lang="ts">
 const props = defineProps<{
   sourceId: string
+  showAllColumns?: boolean
 }>()
 </script>
 
 <template>
   <div class="w-full h-full !p-0">
-    <ErdView :source-id="props.sourceId" />
+    <ErdView :source-id="props.sourceId" :show-all-columns="props.showAllColumns" />
   </div>
 </template>
diff --git a/packages/nc-gui/components/dlg/ProjectErd.vue b/packages/nc-gui/components/dlg/ProjectErd.vue
index 5c18099f25..aeac8bc522 100644
--- a/packages/nc-gui/components/dlg/ProjectErd.vue
+++ b/packages/nc-gui/components/dlg/ProjectErd.vue
@@ -46,7 +46,7 @@ onMounted(async () => {
 <template>
   <GeneralModal v-model:visible="isOpen" size="large">
     <div class="h-[80vh]">
-      <ErdView v-if="!isLoading" :source-id="activeSourceId" :base-id="baseId" />
+      <ErdView v-if="!isLoading" :source-id="activeSourceId" :base-id="baseId" :show-all-columns="false" />
     </div>
   </GeneralModal>
 </template>
diff --git a/packages/nc-gui/components/smartsheet/details/Erd.vue b/packages/nc-gui/components/smartsheet/details/Erd.vue
index 1942a4fd1e..2e182656cc 100644
--- a/packages/nc-gui/components/smartsheet/details/Erd.vue
+++ b/packages/nc-gui/components/smartsheet/details/Erd.vue
@@ -24,7 +24,7 @@ const indicator = h(LoadingOutlined, {
     </div>
 
     <Suspense v-else>
-      <LazyErdView :table="activeTable" :source-id="activeTable?.source_id" :show-all-columns="false" />
+      <LazyErdView :table="activeTable" :source-id="activeTable?.source_id" show-all-columns />
       <template #fallback>
         <div class="h-full w-full flex flex-col justify-center items-center mt-28">
           <a-spin size="large" :indicator="indicator" />

From fcbf511bfdeb72917538f434ee6b9f285b6b4c49 Mon Sep 17 00:00:00 2001
From: Muhammed Mustafa <mustafa@nocodb.com>
Date: Mon, 23 Oct 2023 08:49:26 +0000
Subject: [PATCH 2/2] fix: Fixed test for ERD

---
 tests/playwright/tests/db/features/erd.spec.ts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/tests/playwright/tests/db/features/erd.spec.ts b/tests/playwright/tests/db/features/erd.spec.ts
index 6ee43e189b..3dca369431 100644
--- a/tests/playwright/tests/db/features/erd.spec.ts
+++ b/tests/playwright/tests/db/features/erd.spec.ts
@@ -61,6 +61,8 @@ test.describe('Erd', () => {
 
     const erd = dashboard.details.relations;
 
+    await erd.clickShowColumnNames();
+
     if (isPg(context)) {
       await erd.verifyNodesCount(sakilaTables.length);
       await erd.verifyEdgesCount({
@@ -181,7 +183,6 @@ test.describe('Erd', () => {
     await openErdOfATable('Country');
     const erd = dashboard.details.relations;
 
-    await erd.clickShowColumnNames();
     // Verify tables with default config
     await erd.verifyColumns({
       tableName: `country`,
@@ -224,7 +225,6 @@ test.describe('Erd', () => {
     // Verify
     await dashboard.grid.topbar.btn_details.click();
     await openErdOfATable('Country');
-    await erd.clickShowColumnNames();
 
     await erd.verifyNode({
       tableName: `country`,
@@ -245,7 +245,6 @@ test.describe('Erd', () => {
     // Verify
     await dashboard.grid.topbar.btn_details.click();
     await openErdOfATable('Country');
-    await erd.clickShowColumnNames();
 
     await erd.verifyNode({
       tableName: `country`,