Some updates, deleting external tables does delete data so warn about this.
This commit is contained in:
parent
d04ce5c678
commit
fdf82a6276
|
@ -150,24 +150,23 @@
|
||||||
if (!source) {
|
if (!source) {
|
||||||
return ""
|
return ""
|
||||||
}
|
}
|
||||||
|
const screenCount = affectedScreens.length
|
||||||
let message = `Removing ${source?.name} `
|
let message = `Removing ${source?.name} `
|
||||||
let initialLength = message.length
|
let initialLength = message.length
|
||||||
if (sourceType === SourceType.TABLE) {
|
if (sourceType === SourceType.TABLE) {
|
||||||
const views = "views" in source ? Object.values(source?.views ?? []) : []
|
const views = "views" in source ? Object.values(source?.views ?? []) : []
|
||||||
if (isInternalTable) {
|
message += `will delete its data${
|
||||||
message += `will delete its data${
|
views.length
|
||||||
views.length ? `, views (${views.length})` : ""
|
? `${screenCount ? "," : " and"} views (${views.length})`
|
||||||
}`
|
: ""
|
||||||
} else if (views.length) {
|
}`
|
||||||
message += `will delete its views (${views.length})`
|
|
||||||
}
|
|
||||||
} else if (sourceType === SourceType.DATASOURCE) {
|
} else if (sourceType === SourceType.DATASOURCE) {
|
||||||
const queryList = getDatasourceQueries()
|
const queryList = getDatasourceQueries()
|
||||||
if (queryList.length) {
|
if (queryList.length) {
|
||||||
message += `will delete its queries (${queryList.length})`
|
message += `will delete its queries (${queryList.length})`
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (affectedScreens.length) {
|
if (screenCount) {
|
||||||
message +=
|
message +=
|
||||||
initialLength !== message.length
|
initialLength !== message.length
|
||||||
? ", and break connected screens:"
|
? ", and break connected screens:"
|
||||||
|
@ -187,16 +186,20 @@
|
||||||
title={`Are you sure you want to delete this ${sourceType}?`}
|
title={`Are you sure you want to delete this ${sourceType}?`}
|
||||||
>
|
>
|
||||||
<div class="content">
|
<div class="content">
|
||||||
{#if affectedScreens.length > 0 && sourceType}
|
{#if sourceType}
|
||||||
<p class="warning">
|
<p class="warning">
|
||||||
{buildMessage(sourceType)}
|
{buildMessage(sourceType)}
|
||||||
<span class="screens">
|
{#if affectedScreens.length > 0}
|
||||||
{#each affectedScreens as item, idx}
|
<span class="screens">
|
||||||
<Link overBackground target="_blank" href={item.url}
|
{#each affectedScreens as item, idx}
|
||||||
>{item.text}{idx !== affectedScreens.length - 1 ? "," : ""}</Link
|
<Link overBackground target="_blank" href={item.url}
|
||||||
>
|
>{item.text}{idx !== affectedScreens.length - 1
|
||||||
{/each}
|
? ","
|
||||||
</span>
|
: ""}</Link
|
||||||
|
>
|
||||||
|
{/each}
|
||||||
|
</span>
|
||||||
|
{/if}
|
||||||
</p>
|
</p>
|
||||||
{/if}
|
{/if}
|
||||||
<p class="warning">
|
<p class="warning">
|
||||||
|
|
Loading…
Reference in New Issue