Fix secondary links default value and types (#8532)

Fixes broken companies view

packages/twenty-server/src/engine/api/__mocks__/object-metadata-item.mock.ts

@@ -93,7 +93,7 @@ const fieldLinksMock = {
   type: FieldMetadataType.LINKS,
   isNullable: false,
   defaultValue: [
-    { primaryLinkLabel: '', primaryLinkUrl: '', secondaryLinks: {} },
+    { primaryLinkLabel: '', primaryLinkUrl: '', secondaryLinks: [] },
   ],
 };
 

packages/twenty-server/src/engine/core-modules/calendar/dtos/timeline-calendar-event.dto.ts

@@ -22,7 +22,7 @@ export class LinksMetadata {
   primaryLinkUrl: string;
 
   @Field(() => [LinkMetadata], { nullable: true })
-  secondaryLinks: object | null;
+  secondaryLinks: LinkMetadata[] | null;
 }
 
 @ObjectType('TimelineCalendarEvent')

packages/twenty-server/src/engine/metadata-modules/field-metadata/composite-types/links.composite-type.ts

@@ -27,8 +27,13 @@ export const linksCompositeType: CompositeType = {
   ],
 };
 
+export type LinkMetadata = {
+  label: string;
+  url: string;
+};
+
 export type LinksMetadata = {
   primaryLinkLabel: string;
   primaryLinkUrl: string;
-  secondaryLinks: object | null;
+  secondaryLinks: LinkMetadata[] | null;
 };

packages/twenty-server/src/engine/metadata-modules/field-metadata/dtos/default-value.input.ts

@@ -31,7 +31,7 @@ export class FieldMetadataDefaultValueString {
 
 export class FieldMetadataDefaultValueRawJson {
   @ValidateIf((_object, value) => value !== null)
-  @IsObject()
+  @IsObject() // TODO: Should this also allow arrays?
   value: object | null;
 }
 
@@ -137,6 +137,14 @@ export class FieldMetadataDefaultValueAddress {
   addressLng: number | null;
 }
 
+class LinkMetadata {
+  @IsString()
+  label: string;
+
+  @IsString()
+  url: string;
+}
+
 export class FieldMetadataDefaultValueLinks {
   @ValidateIf((_object, value) => value !== null)
   @IsQuotedString()
@@ -147,8 +155,8 @@ export class FieldMetadataDefaultValueLinks {
   primaryLinkUrl: string | null;
 
   @ValidateIf((_object, value) => value !== null)
-  @IsObject()
-  secondaryLinks: object | null;
+  @IsArray()
+  secondaryLinks: LinkMetadata[] | null;
 }
 
 export class FieldMetadataDefaultActor {

packages/twenty-server/src/engine/metadata-modules/field-metadata/utils/generate-default-value.ts

@@ -38,7 +38,7 @@ export function generateDefaultValue(
       return {
         primaryLinkLabel: "''",
         primaryLinkUrl: "''",
-        secondaryLinks: [],
+        secondaryLinks: "'[]'",
       };
     case FieldMetadataType.PHONES:
       return {