fix: empty string as default value for string types (#2691)

* fix: empty string as default value for string types

* Fixes

---------

Co-authored-by: Charles Bochet <charles@twenty.com>
This commit is contained in:
Jérémy M
2023-11-24 16:30:28 +01:00
committed by GitHub
parent 0376d0b7e0
commit 2b597d817c
6 changed files with 23 additions and 9 deletions

View File

@ -36,7 +36,7 @@ export function convertFieldMetadataToColumnActions(
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.value, columnName: fieldMetadata.targetColumnMap.value,
columnType: 'text', columnType: 'text',
defaultValue: serializeDefaultValue(defaultValue?.value), defaultValue: serializeDefaultValue(defaultValue?.value ?? ''),
}, },
]; ];
} }
@ -52,7 +52,7 @@ export function convertFieldMetadataToColumnActions(
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.value, columnName: fieldMetadata.targetColumnMap.value,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.value), defaultValue: serializeDefaultValue(defaultValue?.value ?? ''),
}, },
]; ];
} }
@ -120,13 +120,13 @@ export function convertFieldMetadataToColumnActions(
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.label, columnName: fieldMetadata.targetColumnMap.label,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.label), defaultValue: serializeDefaultValue(defaultValue?.label ?? ''),
}, },
{ {
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.url, columnName: fieldMetadata.targetColumnMap.url,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.url), defaultValue: serializeDefaultValue(defaultValue?.url ?? ''),
}, },
]; ];
} }
@ -146,7 +146,7 @@ export function convertFieldMetadataToColumnActions(
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.currencyCode, columnName: fieldMetadata.targetColumnMap.currencyCode,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.currencyCode), defaultValue: serializeDefaultValue(defaultValue?.currencyCode ?? ''),
}, },
]; ];
} }
@ -159,13 +159,13 @@ export function convertFieldMetadataToColumnActions(
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.firstName, columnName: fieldMetadata.targetColumnMap.firstName,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.firstName), defaultValue: serializeDefaultValue(defaultValue?.firstName ?? ''),
}, },
{ {
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
columnName: fieldMetadata.targetColumnMap.lastName, columnName: fieldMetadata.targetColumnMap.lastName,
columnType: 'varchar', columnType: 'varchar',
defaultValue: serializeDefaultValue(defaultValue?.lastName), defaultValue: serializeDefaultValue(defaultValue?.lastName ?? ''),
}, },
]; ];
} }

View File

@ -39,21 +39,25 @@ export const addCompanyTable: WorkspaceMigrationTableAction[] = [
columnName: 'linkedinLinkUrl', columnName: 'linkedinLinkUrl',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'linkedinLinkLabel', columnName: 'linkedinLinkLabel',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'xLinkUrl', columnName: 'xLinkUrl',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'xLinkLabel', columnName: 'xLinkLabel',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'annualRecurringRevenueAmountMicros', columnName: 'annualRecurringRevenueAmountMicros',

View File

@ -19,7 +19,7 @@ export const addViewFieldTable: WorkspaceMigrationTableAction[] = [
}, },
{ {
columnName: 'position', columnName: 'position',
columnType: 'integer', columnType: 'float',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: 0, defaultValue: 0,
}, },

View File

@ -22,7 +22,7 @@ export const addActivityTable: WorkspaceMigrationTableAction[] = [
columnName: 'body', columnName: 'body',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''", defaultValue: "'{}'",
}, },
{ {
columnName: 'type', columnName: 'type',

View File

@ -16,6 +16,7 @@ export const addApiKeyTable: WorkspaceMigrationTableAction[] = [
columnName: 'name', columnName: 'name',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'revokedAt', columnName: 'revokedAt',

View File

@ -28,46 +28,55 @@ export const addPersonTable: WorkspaceMigrationTableAction[] = [
columnName: 'email', columnName: 'email',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'linkedinLinkUrl', columnName: 'linkedinLinkUrl',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'linkedinLinkLabel', columnName: 'linkedinLinkLabel',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'xLinkUrl', columnName: 'xLinkUrl',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'xLinkLabel', columnName: 'xLinkLabel',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'jobTitle', columnName: 'jobTitle',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'phone', columnName: 'phone',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'city', columnName: 'city',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'avatarUrl', columnName: 'avatarUrl',
columnType: 'varchar', columnType: 'varchar',
action: WorkspaceMigrationColumnActionType.CREATE, action: WorkspaceMigrationColumnActionType.CREATE,
defaultValue: "''",
}, },
{ {
columnName: 'companyId', columnName: 'companyId',