diff --git a/packages/twenty-front/src/modules/object-metadata/contexts/PreComputedChipGeneratorsContext.ts b/packages/twenty-front/src/modules/object-metadata/contexts/PreComputedChipGeneratorsContext.ts index ed7b734bc..1b1df0e70 100644 --- a/packages/twenty-front/src/modules/object-metadata/contexts/PreComputedChipGeneratorsContext.ts +++ b/packages/twenty-front/src/modules/object-metadata/contexts/PreComputedChipGeneratorsContext.ts @@ -8,9 +8,8 @@ export type ChipGeneratorPerObjectNameSingularPerFieldName = Record< Record RecordChipData> >; -export type IdentifierChipGeneratorPerObject = Record< - string, - (record: ObjectRecord) => RecordChipData +export type IdentifierChipGeneratorPerObject = Partial< + Record RecordChipData> >; export type PreComputedChipGeneratorsContextProps = { diff --git a/packages/twenty-front/src/modules/object-metadata/utils/generateDefaultRecordChipData.ts b/packages/twenty-front/src/modules/object-metadata/utils/generateDefaultRecordChipData.ts index 5b3acd869..0698f587d 100644 --- a/packages/twenty-front/src/modules/object-metadata/utils/generateDefaultRecordChipData.ts +++ b/packages/twenty-front/src/modules/object-metadata/utils/generateDefaultRecordChipData.ts @@ -1,15 +1,25 @@ +import { RecordChipData } from '@/object-record/record-field/types/RecordChipData'; import { isFieldFullNameValue } from '@/object-record/record-field/types/guards/isFieldFullNameValue'; import { ObjectRecord } from '@/object-record/types/ObjectRecord'; -export const generateDefaultRecordChipData = (record: ObjectRecord) => { +type GenerateDefaultRecordChipDataArgs = { + record: ObjectRecord; + objectNameSingular: string; +}; +export const generateDefaultRecordChipData = ({ + objectNameSingular, + record, +}: GenerateDefaultRecordChipDataArgs): RecordChipData => { const name = isFieldFullNameValue(record.name) - ? record.name.firstName + ' ' + record.name.lastName + ? `${record.name.firstName} ${record.name.lastName}` : (record.name ?? ''); return { - name, - avatarUrl: name, avatarType: 'rounded', - linkToShowPage: false, + avatarUrl: name, + isLabelIdentifier: false, + name, + objectNameSingular, + recordId: record.id, }; }; diff --git a/packages/twenty-front/src/modules/object-record/hooks/useRecordChipData.ts b/packages/twenty-front/src/modules/object-record/hooks/useRecordChipData.ts index 7ade7cb90..e3ee43edb 100644 --- a/packages/twenty-front/src/modules/object-record/hooks/useRecordChipData.ts +++ b/packages/twenty-front/src/modules/object-record/hooks/useRecordChipData.ts @@ -1,24 +1,37 @@ import { PreComputedChipGeneratorsContext } from '@/object-metadata/contexts/PreComputedChipGeneratorsContext'; import { generateDefaultRecordChipData } from '@/object-metadata/utils/generateDefaultRecordChipData'; +import { RecordChipData } from '@/object-record/record-field/types/RecordChipData'; import { ObjectRecord } from '@/object-record/types/ObjectRecord'; import { useContext } from 'react'; +import { isDefined } from 'twenty-shared'; +type UseRecordChipDataArgs = { + objectNameSingular: string; + record: ObjectRecord; +}; +type UseRecordChipDataReturnType = { + recordChipData: RecordChipData; +}; export const useRecordChipData = ({ objectNameSingular, record, -}: { - objectNameSingular: string; - record: ObjectRecord; -}) => { +}: UseRecordChipDataArgs): UseRecordChipDataReturnType => { const { identifierChipGeneratorPerObject } = useContext( PreComputedChipGeneratorsContext, ); - const generateRecordChipData = - identifierChipGeneratorPerObject[objectNameSingular] ?? - generateDefaultRecordChipData; + const identifierChipGenerator = + identifierChipGeneratorPerObject[objectNameSingular]; + if (isDefined(identifierChipGenerator)) { + return { + recordChipData: identifierChipGenerator(record), + }; + } - const recordChipData = generateRecordChipData(record); - - return { recordChipData }; + return { + recordChipData: generateDefaultRecordChipData({ + objectNameSingular, + record, + }), + }; }; diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/display/components/RelationFromManyFieldDisplay.tsx b/packages/twenty-front/src/modules/object-record/record-field/meta-types/display/components/RelationFromManyFieldDisplay.tsx index f238be043..337e2b445 100644 --- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/display/components/RelationFromManyFieldDisplay.tsx +++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/display/components/RelationFromManyFieldDisplay.tsx @@ -6,7 +6,7 @@ import { RecordChip } from '@/object-record/components/RecordChip'; import { useFieldFocus } from '@/object-record/record-field/hooks/useFieldFocus'; import { useRelationFromManyFieldDisplay } from '@/object-record/record-field/meta-types/hooks/useRelationFromManyFieldDisplay'; import { ExpandableList } from '@/ui/layout/expandable-list/components/ExpandableList'; -import { isNull } from '@sniptt/guards'; +import { isDefined } from 'twenty-shared'; export const RelationFromManyFieldDisplay = () => { const { fieldValue, fieldDefinition } = useRelationFromManyFieldDisplay(); @@ -48,43 +48,37 @@ export const RelationFromManyFieldDisplay = () => { return ( - {fieldValue - .filter((record) => !isNull(record[relationFieldName])) - .map((record) => ( - - ))} + {fieldValue.filter(isDefined).map((record) => ( + + ))} ); } else if (isRelationFromActivityTargets) { return ( - {activityTargetObjectRecords - .filter((record) => !isNull(record.targetObject)) - .map((record) => ( - - ))} + {activityTargetObjectRecords.filter(isDefined).map((record) => ( + + ))} ); } else { return ( - {fieldValue - .filter((record) => !isNull(record)) - .map((record) => ( - - ))} + {fieldValue.filter(isDefined).map((record) => ( + + ))} ); } diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationFromManyFieldDisplay.ts b/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationFromManyFieldDisplay.ts index 86c389d0b..42f851ff1 100644 --- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationFromManyFieldDisplay.ts +++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationFromManyFieldDisplay.ts @@ -44,14 +44,26 @@ export const useRelationFromManyFieldDisplay = () => { ? maxWidth - FIELD_EDIT_BUTTON_WIDTH : maxWidth; - if (!isNonEmptyString(fieldDefinition.metadata.objectMetadataNameSingular)) { + if ( + !isDefined(fieldDefinition.metadata.objectMetadataNameSingular) || + !isNonEmptyString(fieldDefinition.metadata.objectMetadataNameSingular) + ) { throw new Error('Object metadata name singular is not a non-empty string'); } - const generateRecordChipData = + const fieldChipGenerator = chipGeneratorPerObjectPerField[ fieldDefinition.metadata.objectMetadataNameSingular - ]?.[fieldDefinition.metadata.fieldName] ?? generateDefaultRecordChipData; + ]?.[fieldDefinition.metadata.fieldName]; + const generateRecordChipData = isDefined(fieldChipGenerator) + ? fieldChipGenerator + : (record: ObjectRecord) => + generateDefaultRecordChipData({ + record, + // @ts-expect-error Above assertions does not infer that fieldDefinition.metadata.objectMetadataNameSingular always defined + objectNameSingular: + fieldDefinition.metadata.objectMetadataNameSingular, + }); return { fieldDefinition, diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationToOneFieldDisplay.ts b/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationToOneFieldDisplay.ts index 3ad94bb3c..d40fb7a28 100644 --- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationToOneFieldDisplay.ts +++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/hooks/useRelationToOneFieldDisplay.ts @@ -2,13 +2,13 @@ import { isNonEmptyString } from '@sniptt/guards'; import { useContext } from 'react'; import { PreComputedChipGeneratorsContext } from '@/object-metadata/contexts/PreComputedChipGeneratorsContext'; -import { generateDefaultRecordChipData } from '@/object-metadata/utils/generateDefaultRecordChipData'; import { useRecordFieldValue } from '@/object-record/record-store/contexts/RecordFieldValueSelectorContext'; import { ObjectRecord } from '@/object-record/types/ObjectRecord'; import { FIELD_EDIT_BUTTON_WIDTH } from '@/ui/field/display/constants/FieldEditButtonWidth'; import { isDefined } from 'twenty-shared'; import { FieldMetadataType } from '~/generated-metadata/graphql'; +import { generateDefaultRecordChipData } from '@/object-metadata/utils/generateDefaultRecordChipData'; import { FieldContext } from '../../contexts/FieldContext'; import { assertFieldMetadata } from '../../types/guards/assertFieldMetadata'; import { isFieldRelation } from '../../types/guards/isFieldRelation'; @@ -44,14 +44,26 @@ export const useRelationToOneFieldDisplay = () => { ? maxWidth - FIELD_EDIT_BUTTON_WIDTH : maxWidth; - if (!isNonEmptyString(fieldDefinition.metadata.objectMetadataNameSingular)) { + if ( + !isDefined(fieldDefinition.metadata.objectMetadataNameSingular) || + !isNonEmptyString(fieldDefinition.metadata.objectMetadataNameSingular) + ) { throw new Error('Object metadata name singular is not a non-empty string'); } - const generateRecordChipData = + const fieldChipGenerator = chipGeneratorPerObjectPerField[ fieldDefinition.metadata.objectMetadataNameSingular - ]?.[fieldDefinition.metadata.fieldName] ?? generateDefaultRecordChipData; + ]?.[fieldDefinition.metadata.fieldName]; + const generateRecordChipData = isDefined(fieldChipGenerator) + ? fieldChipGenerator + : (record: ObjectRecord) => + generateDefaultRecordChipData({ + record, + // @ts-expect-error Above assertions does not infer that fieldDefinition.metadata.objectMetadataNameSingular always defined + objectNameSingular: + fieldDefinition.metadata.objectMetadataNameSingular, + }); return { fieldDefinition,