Skip to content

Commit 71ba021

Browse files
Merge pull request #921 from contentstack/bugfix/cmg-788
refactor: Enhance content mapping logic by ensuring safeField.id is s…
2 parents 2a2530a + 00b81fa commit 71ba021

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

api/src/services/contentMapper.service.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ const putTestData = async (req: Request) => {
103103
safeField?.id ?
104104
safeField.id.replace(/[{}]/g, '').toLowerCase()
105105
: uuidv4();
106+
safeField.id = id;
106107

107108
fieldIds.push(id);
108109

ui/src/components/ContentMapper/index.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1798,11 +1798,14 @@ const ContentMapper = forwardRef(({ handleStepChange }: contentMapperProps, ref:
17981798
...option,
17991799
isDisabled: selectedOptions?.includes?.(option?.label ?? '')
18001800
}));
1801+
1802+
const isTypeMatch = checkConditions(Fields[data?.contentstackFieldType]?.type?.toLowerCase(), existingField[data?.backupFieldUid]?.value, data);
1803+
18011804
return (
18021805
<div className="table-row">
18031806
<div className="select">
18041807
<Select
1805-
value={(OptionsForRow?.length === 0 || (Fields[data?.contentstackFieldType]?.type?.toLowerCase() !== existingField[data?.backupFieldUid]?.value?.data_type || existingField?.[data?.backupFieldUid]?.label === undefined)) ? OptionValue :
1808+
value={(OptionsForRow?.length === 0 || (!isTypeMatch || existingField?.[data?.backupFieldUid]?.label === undefined)) ? OptionValue :
18061809

18071810
existingField[data?.backupFieldUid]}
18081811
onChange={(selectedOption: FieldTypes) => {
@@ -1815,7 +1818,7 @@ const ContentMapper = forwardRef(({ handleStepChange }: contentMapperProps, ref:
18151818
placeholder="Select Field"
18161819
version={'v2'}
18171820
maxWidth="290px"
1818-
isClearable={data?.backupFieldType === existingField[data?.backupFieldUid]?.value?.data_type && selectedOptions?.includes?.(existingField?.[data?.backupFieldUid]?.label ?? '')}
1821+
isClearable={isTypeMatch && selectedOptions?.includes?.(existingField?.[data?.backupFieldUid]?.label ?? '')}
18191822
options={adjustedOptions}
18201823
isDisabled={OptionValue?.isDisabled || newMigrationData?.project_current_step > 4}
18211824
menuPlacement="auto"

0 commit comments

Comments
 (0)