Merge branch 'master' into BUDI-8279/time-constrains

This commit is contained in:
Adria Navarro 2024-05-23 15:33:50 +02:00 committed by GitHub
commit cf43530e1c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 31 additions and 19 deletions

View File

@ -612,27 +612,29 @@
/> />
</div> </div>
</div> </div>
{#if datasource?.source !== SourceName.ORACLE && datasource?.source !== SourceName.SQL_SERVER && !editableColumn.dateOnly} {#if !editableColumn.timeOnly}
<div> {#if datasource?.source !== SourceName.ORACLE && datasource?.source !== SourceName.SQL_SERVER && !editableColumn.dateOnly}
<div class="row"> <div>
<Label>Time zones</Label> <div class="row">
<AbsTooltip <Label>Time zones</Label>
position="top" <AbsTooltip
type="info" position="top"
text={isCreating type="info"
? null text={isCreating
: "We recommend not changing how timezones are handled for existing columns, as existing data will not be updated"} ? null
> : "We recommend not changing how timezones are handled for existing columns, as existing data will not be updated"}
<Icon size="XS" name="InfoOutline" /> >
</AbsTooltip> <Icon size="XS" name="InfoOutline" />
</AbsTooltip>
</div>
<Toggle
bind:value={editableColumn.ignoreTimezones}
text="Ignore time zones"
/>
</div> </div>
<Toggle {/if}
bind:value={editableColumn.ignoreTimezones} <Toggle bind:value={editableColumn.dateOnly} text="Date only" />
text="Ignore time zones"
/>
</div>
{/if} {/if}
<Toggle bind:value={editableColumn.dateOnly} text="Date only" />
{:else if editableColumn.type === FieldType.NUMBER && !editableColumn.autocolumn} {:else if editableColumn.type === FieldType.NUMBER && !editableColumn.autocolumn}
<div class="split-label"> <div class="split-label">
<div class="label-length"> <div class="label-length">

View File

@ -73,6 +73,16 @@ function validate(table: Table, oldTable?: Table) {
`Column "${key}" has subtype "${column.subtype}" - this is not supported.` `Column "${key}" has subtype "${column.subtype}" - this is not supported.`
) )
} }
if (column.type === FieldType.DATETIME) {
const oldColumn = oldTable?.schema[key] as typeof column
if (oldColumn && column.timeOnly !== oldColumn.timeOnly) {
throw new Error(
`Column "${key}" can not change from time to datetime or viceversa.`
)
}
}
} }
} }