Skip to content
This repository was archived by the owner on Dec 6, 2024. It is now read-only.

Commit 8d145e2

Browse files
authored
Merge pull request nocodb#6782 from nocodb/nc-fix/hide-col-for-project-erd
Hide col for project erd
2 parents 3ee2d57 + fcbf511 commit 8d145e2

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

packages/nc-gui/components/dashboard/settings/DataSources.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -554,7 +554,7 @@ const isEditBaseModalOpen = computed({
554554
/>
555555
<GeneralModal v-model:visible="isErdModalOpen" size="large">
556556
<div class="h-[80vh]">
557-
<LazyDashboardSettingsErd :source-id="activeBaseId" />
557+
<LazyDashboardSettingsErd :source-id="activeBaseId" :show-all-columns="false" />
558558
</div>
559559
</GeneralModal>
560560
<GeneralModal v-model:visible="isMetaDataModal" size="medium">
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
<script setup lang="ts">
22
const props = defineProps<{
33
sourceId: string
4+
showAllColumns?: boolean
45
}>()
56
</script>
67

78
<template>
89
<div class="w-full h-full !p-0">
9-
<ErdView :source-id="props.sourceId" />
10+
<ErdView :source-id="props.sourceId" :show-all-columns="props.showAllColumns" />
1011
</div>
1112
</template>

packages/nc-gui/components/dlg/ProjectErd.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ onMounted(async () => {
4646
<template>
4747
<GeneralModal v-model:visible="isOpen" size="large">
4848
<div class="h-[80vh]">
49-
<ErdView v-if="!isLoading" :source-id="activeSourceId" :base-id="baseId" />
49+
<ErdView v-if="!isLoading" :source-id="activeSourceId" :base-id="baseId" :show-all-columns="false" />
5050
</div>
5151
</GeneralModal>
5252
</template>

packages/nc-gui/components/smartsheet/details/Erd.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const indicator = h(LoadingOutlined, {
2424
</div>
2525

2626
<Suspense v-else>
27-
<LazyErdView :table="activeTable" :source-id="activeTable?.source_id" :show-all-columns="false" />
27+
<LazyErdView :table="activeTable" :source-id="activeTable?.source_id" show-all-columns />
2828
<template #fallback>
2929
<div class="h-full w-full flex flex-col justify-center items-center mt-28">
3030
<a-spin size="large" :indicator="indicator" />

tests/playwright/tests/db/features/erd.spec.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ test.describe('Erd', () => {
6161

6262
const erd = dashboard.details.relations;
6363

64+
await erd.clickShowColumnNames();
65+
6466
if (isPg(context)) {
6567
await erd.verifyNodesCount(sakilaTables.length);
6668
await erd.verifyEdgesCount({
@@ -181,7 +183,6 @@ test.describe('Erd', () => {
181183
await openErdOfATable('Country');
182184
const erd = dashboard.details.relations;
183185

184-
await erd.clickShowColumnNames();
185186
// Verify tables with default config
186187
await erd.verifyColumns({
187188
tableName: `country`,
@@ -224,7 +225,6 @@ test.describe('Erd', () => {
224225
// Verify
225226
await dashboard.grid.topbar.btn_details.click();
226227
await openErdOfATable('Country');
227-
await erd.clickShowColumnNames();
228228

229229
await erd.verifyNode({
230230
tableName: `country`,
@@ -245,7 +245,6 @@ test.describe('Erd', () => {
245245
// Verify
246246
await dashboard.grid.topbar.btn_details.click();
247247
await openErdOfATable('Country');
248-
await erd.clickShowColumnNames();
249248

250249
await erd.verifyNode({
251250
tableName: `country`,

0 commit comments

Comments
 (0)