Replace shouldSyncNameAndLabel with isLabelSyncedWithName (#8067)

For consistency. This was not deployed yet so allowing myself just to rename everything, meaning developers will need to reset their db.
2024-10-25 18:28:58 +02:00
parent e5175194ac
commit d51a797d91
22 changed files with 80 additions and 80 deletions
--- a/packages/twenty-front/src/modules/command-menu/hooks/tests/useCommandMenu.test.tsx
+++ b/packages/twenty-front/src/modules/command-menu/hooks/tests/useCommandMenu.test.tsx
@ -124,7 +124,7 @@ describe('useCommandMenu', () => {
          namePlural: 'tasks',
          labelSingular: 'Task',
          labelPlural: 'Tasks',
-          shouldSyncLabelAndName: true,
+          isLabelSyncedWithName: true,
          shortcut: 'T',
          description: 'A task',
          icon: 'IconCheckbox',
--- a/packages/twenty-front/src/modules/object-metadata/graphql/queries.ts
+++ b/packages/twenty-front/src/modules/object-metadata/graphql/queries.ts
@ -25,7 +25,7 @@ export const FIND_MANY_OBJECT_METADATA_ITEMS = gql`
          labelIdentifierFieldMetadataId
          imageIdentifierFieldMetadataId
          shortcut
-          shouldSyncLabelAndName
+          isLabelSyncedWithName
          indexMetadatas(paging: { first: 100 }) {
            edges {
              node {
--- a/packages/twenty-front/src/modules/object-metadata/hooks/mocks/useFindManyObjectMetadataItems.ts
+++ b/packages/twenty-front/src/modules/object-metadata/hooks/mocks/useFindManyObjectMetadataItems.ts
@ -25,7 +25,7 @@ export const query = gql`
          labelIdentifierFieldMetadataId
          imageIdentifierFieldMetadataId
          shortcut
-          shouldSyncLabelAndName
+          isLabelSyncedWithName
          fields(paging: { first: 1000 }, filter: $fieldFilter) {
            edges {
              node {
--- a/packages/twenty-front/src/modules/object-metadata/validation-schemas/tests/objectMetadataItemSchema.test.ts
+++ b/packages/twenty-front/src/modules/object-metadata/validation-schemas/tests/objectMetadataItemSchema.test.ts
@ -30,7 +30,7 @@ describe('objectMetadataItemSchema', () => {
      namePlural: 'notCamelCase',
      nameSingular: 'notCamelCase',
      updatedAt: 'invalid date',
-      shouldSyncLabelAndName: 'not a boolean',
+      isLabelSyncedWithName: 'not a boolean',
    };

    // When
--- a/packages/twenty-front/src/modules/object-metadata/validation-schemas/objectMetadataItemSchema.ts
+++ b/packages/twenty-front/src/modules/object-metadata/validation-schemas/objectMetadataItemSchema.ts
@ -27,5 +27,5 @@ export const objectMetadataItemSchema = z.object({
  nameSingular: camelCaseStringSchema,
  updatedAt: z.string().datetime(),
  shortcut: z.string().nullable().optional(),
-  shouldSyncLabelAndName: z.boolean(),
+  isLabelSyncedWithName: z.boolean(),
 }) satisfies z.ZodType<ObjectMetadataItem>;
--- a/packages/twenty-front/src/modules/object-record/object-sort-dropdown/utils/tests/turnSortsIntoOrderBy.test.ts
+++ b/packages/twenty-front/src/modules/object-record/object-sort-dropdown/utils/tests/turnSortsIntoOrderBy.test.ts
@ -25,7 +25,7 @@ const objectMetadataItem: ObjectMetadataItem = {
  isRemote: false,
  labelPlural: 'object1s',
  labelSingular: 'object1',
-  shouldSyncLabelAndName: true,
+  isLabelSyncedWithName: true,
 };

 describe('turnSortsIntoOrderBy', () => {
--- a/packages/twenty-front/src/modules/object-record/relation-picker/hooks/tests/useLimitPerMetadataItem.test.tsx
+++ b/packages/twenty-front/src/modules/object-record/relation-picker/hooks/tests/useLimitPerMetadataItem.test.tsx
@ -26,7 +26,7 @@ describe('useLimitPerMetadataItem', () => {
      namePlural: 'namePlural',
      nameSingular: 'nameSingular',
      updatedAt: 'updatedAt',
-      shouldSyncLabelAndName: false,
+      isLabelSyncedWithName: false,
      fields: [],
      indexMetadatas: [],
    },
--- a/packages/twenty-front/src/modules/object-record/relation-picker/hooks/tests/useMultiObjectSearch.test.tsx
+++ b/packages/twenty-front/src/modules/object-record/relation-picker/hooks/tests/useMultiObjectSearch.test.tsx
@ -34,7 +34,7 @@ const objectData: ObjectMetadataItem[] = [
    labelSingular: 'labelSingular',
    namePlural: 'namePlural',
    nameSingular: 'nameSingular',
-    shouldSyncLabelAndName: false,
+    isLabelSyncedWithName: false,
    updatedAt: 'updatedAt',
    fields: [
      {
--- a/packages/twenty-front/src/modules/settings/data-model/objects/forms/components/SettingsDataModelObjectAboutForm.tsx
+++ b/packages/twenty-front/src/modules/settings/data-model/objects/forms/components/SettingsDataModelObjectAboutForm.tsx
@ -36,7 +36,7 @@ export const settingsDataModelObjectAboutFormSchema = objectMetadataItemSchema
      .pick({
        nameSingular: true,
        namePlural: true,
-        shouldSyncLabelAndName: true,
+        isLabelSyncedWithName: true,
      })
      .partial(),
  );
@ -115,10 +115,10 @@ export const SettingsDataModelObjectAboutForm = ({
    isAdvancedModeEnabled,
  );

-  const shouldSyncLabelAndName = watch('shouldSyncLabelAndName');
+  const isLabelSyncedWithName = watch('isLabelSyncedWithName');
  const labelSingular = watch('labelSingular');
  const labelPlural = watch('labelPlural');
-  const apiNameTooltipText = shouldSyncLabelAndName
+  const apiNameTooltipText = isLabelSyncedWithName
    ? 'Deactivate "Synchronize Objects Labels and API Names" to set a custom API name'
    : 'Input must be in camel case and cannot start with a number';

@ -129,7 +129,7 @@ export const SettingsDataModelObjectAboutForm = ({
    setValue('labelPlural', newLabelPluralValue, {
      shouldDirty: isDefined(labelSingular) ? true : false,
    });
-    if (shouldSyncLabelAndName === true) {
+    if (isLabelSyncedWithName === true) {
      fillNamePluralFromLabelPlural(newLabelPluralValue);
    }
  };
@ -182,7 +182,7 @@ export const SettingsDataModelObjectAboutForm = ({
                onChange={(value) => {
                  onChange(value);
                  fillLabelPlural(value);
-                  if (shouldSyncLabelAndName === true) {
+                  if (isLabelSyncedWithName === true) {
                    fillNameSingularFromLabelSingular(value);
                  }
                }}
@ -204,7 +204,7 @@ export const SettingsDataModelObjectAboutForm = ({
                value={value}
                onChange={(value) => {
                  onChange(value);
-                  if (shouldSyncLabelAndName === true) {
+                  if (isLabelSyncedWithName === true) {
                    fillNamePluralFromLabelPlural(value);
                  }
                }}
@ -251,7 +251,7 @@ export const SettingsDataModelObjectAboutForm = ({
                    placeholder: 'listing',
                    defaultValue: objectMetadataItem?.nameSingular,
                    disabled:
-                      disabled || disableNameEdit || shouldSyncLabelAndName,
+                      disabled || disableNameEdit || isLabelSyncedWithName,
                    tooltip: apiNameTooltipText,
                  },
                  {
@ -260,7 +260,7 @@ export const SettingsDataModelObjectAboutForm = ({
                    placeholder: 'listings',
                    defaultValue: objectMetadataItem?.namePlural,
                    disabled:
-                      disabled || disableNameEdit || shouldSyncLabelAndName,
+                      disabled || disableNameEdit || isLabelSyncedWithName,
                    tooltip: apiNameTooltipText,
                  },
                ].map(
@ -318,10 +318,10 @@ export const SettingsDataModelObjectAboutForm = ({
                  ),
                )}
                <Controller
-                  name="shouldSyncLabelAndName"
+                  name="isLabelSyncedWithName"
                  control={control}
                  defaultValue={
-                    objectMetadataItem?.shouldSyncLabelAndName ?? true
+                    objectMetadataItem?.isLabelSyncedWithName ?? true
                  }
                  render={({ field: { onChange, value } }) => (
                    <SyncObjectLabelAndNameToggle
--- a/packages/twenty-front/src/modules/settings/data-model/validation-schemas/tests/settingsCreateObjectInputSchema.test.ts
+++ b/packages/twenty-front/src/modules/settings/data-model/validation-schemas/tests/settingsCreateObjectInputSchema.test.ts
@ -14,7 +14,7 @@ describe('settingsCreateObjectInputSchema', () => {
      labelSingular: 'Label ',
      namePlural: 'namePlural',
      nameSingular: 'nameSingular',
-      shouldSyncLabelAndName: false,
+      isLabelSyncedWithName: false,
    };

    // When
@ -29,7 +29,7 @@ describe('settingsCreateObjectInputSchema', () => {
      labelSingular: 'Label',
      namePlural: 'namePlural',
      nameSingular: 'nameSingular',
-      shouldSyncLabelAndName: false,
+      isLabelSyncedWithName: false,
    });
  });

--- a/packages/twenty-front/src/modules/settings/data-model/validation-schemas/settingsCreateObjectInputSchema.ts
+++ b/packages/twenty-front/src/modules/settings/data-model/validation-schemas/settingsCreateObjectInputSchema.ts
@ -12,6 +12,6 @@ export const settingsCreateObjectInputSchema =
      namePlural:
        values.namePlural ??
        computeMetadataNameFromLabelOrThrow(values.labelPlural),
-      shouldSyncLabelAndName: values.shouldSyncLabelAndName ?? true,
+      isLabelSyncedWithName: values.isLabelSyncedWithName ?? true,
    }),
  );