From 2d55d90235a61b558e6fa95acebad748f122a8a5 Mon Sep 17 00:00:00 2001
From: Thomas Trompette <thomas.trompette@sfr.fr>
Date: Thu, 16 Jan 2025 18:58:25 +0100
Subject: [PATCH 1/4] Update database event selection

---
 .../utils/generateWorkflowDiagram.ts          |  2 +-
 .../components/WorkflowStepHeader.tsx         |  6 +-
 ...DrawerWorkflowSelectTriggerTypeContent.tsx | 38 +++++++++-
 .../WorkflowEditTriggerDatabaseEventForm.tsx  | 75 ++++---------------
 .../constants/DatabaseTriggerName.ts          |  5 ++
 .../constants/DatabaseTriggerTypes.ts         | 25 +++++++
 .../constants/ObjectEventTriggers.ts          |  8 +-
 .../{TriggerTypes.ts => OtherTriggerTypes.ts} |  9 +--
 .../utils/getTriggerDefaultDefinition.ts      | 17 ++++-
 .../utils/getTriggerStepName.ts               |  4 +-
 10 files changed, 107 insertions(+), 82 deletions(-)
 create mode 100644 packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerName.ts
 create mode 100644 packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
 rename packages/twenty-front/src/modules/workflow/workflow-trigger/constants/{TriggerTypes.ts => OtherTriggerTypes.ts} (62%)

diff --git a/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts b/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
index a760a019cf35..1cdfe811a05f 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
@@ -70,7 +70,7 @@ export const generateWorkflowDiagram = ({
           trigger.settings.eventName,
         );
 
-        triggerLabel = `${capitalize(triggerEvent.objectType)} is ${capitalize(triggerEvent.event)}`;
+        triggerLabel = `Record is ${capitalize(triggerEvent.event)}`;
 
         break;
       }
diff --git a/packages/twenty-front/src/modules/workflow/workflow-steps/components/WorkflowStepHeader.tsx b/packages/twenty-front/src/modules/workflow/workflow-steps/components/WorkflowStepHeader.tsx
index c5fcba10a700..6bd8f30fd0c2 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-steps/components/WorkflowStepHeader.tsx
+++ b/packages/twenty-front/src/modules/workflow/workflow-steps/components/WorkflowStepHeader.tsx
@@ -1,9 +1,9 @@
 import { TextInput } from '@/ui/field/input/components/TextInput';
+import { useTheme } from '@emotion/react';
 import styled from '@emotion/styled';
-import React, { useState } from 'react';
-import { useDebouncedCallback } from 'use-debounce';
 import { IconComponent } from 'packages/twenty-ui';
-import { useTheme } from '@emotion/react';
+import { useState } from 'react';
+import { useDebouncedCallback } from 'use-debounce';
 
 const StyledHeader = styled.div`
   background-color: ${({ theme }) => theme.background.secondary};
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/RightDrawerWorkflowSelectTriggerTypeContent.tsx b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/RightDrawerWorkflowSelectTriggerTypeContent.tsx
index bf8cf359fa58..e4dc9774b856 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/RightDrawerWorkflowSelectTriggerTypeContent.tsx
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/RightDrawerWorkflowSelectTriggerTypeContent.tsx
@@ -3,8 +3,9 @@ import { useRightDrawer } from '@/ui/layout/right-drawer/hooks/useRightDrawer';
 import { RightDrawerPages } from '@/ui/layout/right-drawer/types/RightDrawerPages';
 import { WorkflowWithCurrentVersion } from '@/workflow/types/Workflow';
 import { workflowSelectedNodeState } from '@/workflow/workflow-diagram/states/workflowSelectedNodeState';
+import { DATABASE_TRIGGER_TYPES } from '@/workflow/workflow-trigger/constants/DatabaseTriggerTypes';
+import { OTHER_TRIGGER_TYPES } from '@/workflow/workflow-trigger/constants/OtherTriggerTypes';
 import { TRIGGER_STEP_ID } from '@/workflow/workflow-trigger/constants/TriggerStepId';
-import { TRIGGER_TYPES } from '@/workflow/workflow-trigger/constants/TriggerTypes';
 import { useUpdateWorkflowVersionTrigger } from '@/workflow/workflow-trigger/hooks/useUpdateWorkflowVersionTrigger';
 import { getTriggerDefaultDefinition } from '@/workflow/workflow-trigger/utils/getTriggerDefaultDefinition';
 import styled from '@emotion/styled';
@@ -21,6 +22,14 @@ const StyledActionListContainer = styled.div`
   padding-inline: ${({ theme }) => theme.spacing(2)};
 `;
 
+const StyledSectionTitle = styled.span`
+  color: ${({ theme }) => theme.font.color.tertiary};
+  font-size: ${({ theme }) => theme.font.size.sm};
+  font-weight: ${({ theme }) => theme.font.weight.medium};
+  padding: ${({ theme }) => theme.spacing(2)} ${({ theme }) => theme.spacing(1)}
+    ${({ theme }) => theme.spacing(1)} ${({ theme }) => theme.spacing(1)};
+`;
+
 export const RightDrawerWorkflowSelectTriggerTypeContent = ({
   workflow,
 }: {
@@ -35,14 +44,37 @@ export const RightDrawerWorkflowSelectTriggerTypeContent = ({
 
   return (
     <StyledActionListContainer>
-      {TRIGGER_TYPES.map((action) => (
+      <StyledSectionTitle>Data</StyledSectionTitle>
+      {DATABASE_TRIGGER_TYPES.map((action) => (
+        <MenuItem
+          key={action.name}
+          LeftIcon={action.icon}
+          text={action.name}
+          onClick={async () => {
+            await updateTrigger(
+              getTriggerDefaultDefinition({
+                name: action.name,
+                type: action.type,
+                activeObjectMetadataItems,
+              }),
+            );
+
+            setWorkflowSelectedNode(TRIGGER_STEP_ID);
+
+            openRightDrawer(RightDrawerPages.WorkflowStepEdit);
+          }}
+        />
+      ))}
+      <StyledSectionTitle>Others</StyledSectionTitle>
+      {OTHER_TRIGGER_TYPES.map((action) => (
         <MenuItem
-          key={action.type}
+          key={action.name}
           LeftIcon={action.icon}
           text={action.name}
           onClick={async () => {
             await updateTrigger(
               getTriggerDefaultDefinition({
+                name: action.name,
                 type: action.type,
                 activeObjectMetadataItems,
               }),
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
index 873c5ab2e745..a84f00bd14ec 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
@@ -29,20 +29,15 @@ export const WorkflowEditTriggerDatabaseEventForm = ({
 
   const { activeObjectMetadataItems } = useFilteredObjectMetadataItems();
 
-  const triggerEvent = isDefined(trigger)
-    ? splitWorkflowTriggerEventName(trigger.settings.eventName)
-    : undefined;
+  const triggerEvent = splitWorkflowTriggerEventName(
+    trigger.settings.eventName,
+  );
 
   const availableMetadata: Array<SelectOption<string>> =
     activeObjectMetadataItems.map((item) => ({
       label: item.labelPlural,
       value: item.nameSingular,
     }));
-  const recordTypeMetadata = isDefined(triggerEvent)
-    ? activeObjectMetadataItems.find(
-        (item) => item.nameSingular === triggerEvent.objectType,
-      )
-    : undefined;
 
   const selectedEvent = isDefined(triggerEvent)
     ? OBJECT_EVENT_TRIGGERS.find(
@@ -52,12 +47,12 @@ export const WorkflowEditTriggerDatabaseEventForm = ({
 
   const headerTitle = isDefined(trigger.name)
     ? trigger.name
-    : isDefined(recordTypeMetadata) && isDefined(selectedEvent)
-      ? `When a ${recordTypeMetadata.labelSingular} is ${selectedEvent.label}`
+    : isDefined(selectedEvent)
+      ? selectedEvent.label
       : '-';
 
   const headerType = isDefined(selectedEvent)
-    ? `Trigger · Record is ${selectedEvent.label}`
+    ? `Trigger · ${selectedEvent.label}`
     : '-';
 
   return (
@@ -92,57 +87,13 @@ export const WorkflowEditTriggerDatabaseEventForm = ({
               return;
             }
 
-            triggerOptions.onTriggerUpdate(
-              isDefined(trigger) && isDefined(triggerEvent)
-                ? {
-                    ...trigger,
-                    settings: {
-                      ...trigger.settings,
-                      eventName: `${updatedRecordType}.${triggerEvent.event}`,
-                    },
-                  }
-                : {
-                    name: headerTitle,
-                    type: 'DATABASE_EVENT',
-                    settings: {
-                      eventName: `${updatedRecordType}.${OBJECT_EVENT_TRIGGERS[0].value}`,
-                      outputSchema: {},
-                    },
-                  },
-            );
-          }}
-        />
-        <Select
-          dropdownId="workflow-edit-trigger-event-type"
-          label="Event type"
-          fullWidth
-          value={triggerEvent?.event}
-          emptyOption={{ label: 'Select an option', value: '' }}
-          options={OBJECT_EVENT_TRIGGERS}
-          disabled={triggerOptions.readonly}
-          onChange={(updatedEvent) => {
-            if (triggerOptions.readonly === true) {
-              return;
-            }
-
-            triggerOptions.onTriggerUpdate(
-              isDefined(trigger) && isDefined(triggerEvent)
-                ? {
-                    ...trigger,
-                    settings: {
-                      ...trigger.settings,
-                      eventName: `${triggerEvent.objectType}.${updatedEvent}`,
-                    },
-                  }
-                : {
-                    name: headerTitle,
-                    type: 'DATABASE_EVENT',
-                    settings: {
-                      eventName: `${availableMetadata?.[0].value}.${updatedEvent}`,
-                      outputSchema: {},
-                    },
-                  },
-            );
+            triggerOptions.onTriggerUpdate({
+              ...trigger,
+              settings: {
+                ...trigger.settings,
+                eventName: `${updatedRecordType}.${triggerEvent.event}`,
+              },
+            });
           }}
         />
       </WorkflowStepBody>
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerName.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerName.ts
new file mode 100644
index 000000000000..0aea82a7583e
--- /dev/null
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerName.ts
@@ -0,0 +1,5 @@
+export enum DatabaseTriggerName {
+  RECORD_IS_CREATED = 'Record is Created',
+  RECORD_IS_UPDATED = 'Record is Updated',
+  RECORD_IS_DELETED = 'Record is Deleted',
+}
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
new file mode 100644
index 000000000000..fef99520cc1d
--- /dev/null
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
@@ -0,0 +1,25 @@
+import { WorkflowTriggerType } from '@/workflow/types/Workflow';
+import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
+import { IconClick, IconComponent, IconPlus, IconTrash } from 'twenty-ui';
+
+export const DATABASE_TRIGGER_TYPES: Array<{
+  name: DatabaseTriggerName;
+  type: WorkflowTriggerType;
+  icon: IconComponent;
+}> = [
+  {
+    name: DatabaseTriggerName.RECORD_IS_CREATED,
+    type: 'DATABASE_EVENT',
+    icon: IconPlus,
+  },
+  {
+    name: DatabaseTriggerName.RECORD_IS_UPDATED,
+    type: 'DATABASE_EVENT',
+    icon: IconTrash,
+  },
+  {
+    name: DatabaseTriggerName.RECORD_IS_DELETED,
+    type: 'DATABASE_EVENT',
+    icon: IconClick,
+  },
+];
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts
index bd204f7c8fd5..3e7985faa783 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts
@@ -1,16 +1,18 @@
 import { SelectOption } from '@/ui/input/components/Select';
 
+import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
+
 export const OBJECT_EVENT_TRIGGERS: Array<SelectOption<string>> = [
   {
-    label: 'Created',
+    label: DatabaseTriggerName.RECORD_IS_CREATED,
     value: 'created',
   },
   {
-    label: 'Updated',
+    label: DatabaseTriggerName.RECORD_IS_UPDATED,
     value: 'updated',
   },
   {
-    label: 'Deleted',
+    label: DatabaseTriggerName.RECORD_IS_DELETED,
     value: 'deleted',
   },
 ];
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/TriggerTypes.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/OtherTriggerTypes.ts
similarity index 62%
rename from packages/twenty-front/src/modules/workflow/workflow-trigger/constants/TriggerTypes.ts
rename to packages/twenty-front/src/modules/workflow/workflow-trigger/constants/OtherTriggerTypes.ts
index 2360703fbfe3..95d1022be790 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/TriggerTypes.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/OtherTriggerTypes.ts
@@ -1,18 +1,13 @@
 import { WorkflowTriggerType } from '@/workflow/types/Workflow';
 import { IconComponent, IconSettingsAutomation } from 'twenty-ui';
 
-export const TRIGGER_TYPES: Array<{
+export const OTHER_TRIGGER_TYPES: Array<{
   name: string;
   type: WorkflowTriggerType;
   icon: IconComponent;
 }> = [
   {
-    name: 'Database Event',
-    type: 'DATABASE_EVENT',
-    icon: IconSettingsAutomation,
-  },
-  {
-    name: 'Manual Trigger',
+    name: 'Launch manually',
     type: 'MANUAL',
     icon: IconSettingsAutomation,
   },
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
index 091ee91f2cf1..78e2af9b9903 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
@@ -4,13 +4,20 @@ import {
   WorkflowTriggerType,
 } from '@/workflow/types/Workflow';
 import { assertUnreachable } from '@/workflow/utils/assertUnreachable';
+import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
 import { OBJECT_EVENT_TRIGGERS } from '@/workflow/workflow-trigger/constants/ObjectEventTriggers';
 import { getManualTriggerDefaultSettings } from '@/workflow/workflow-trigger/utils/getManualTriggerDefaultSettings';
 
+const isDatabaseName = (name: string): name is DatabaseTriggerName => {
+  return (name as DatabaseTriggerName) !== undefined;
+};
+
 export const getTriggerDefaultDefinition = ({
+  name,
   type,
   activeObjectMetadataItems,
 }: {
+  name: string;
   type: WorkflowTriggerType;
   activeObjectMetadataItems: ObjectMetadataItem[];
 }): WorkflowTrigger => {
@@ -22,10 +29,18 @@ export const getTriggerDefaultDefinition = ({
 
   switch (type) {
     case 'DATABASE_EVENT': {
+      if (!isDatabaseName(name)) {
+        throw new Error('Database event has an invalid name');
+      }
+
       return {
         type,
         settings: {
-          eventName: `${activeObjectMetadataItems[0].nameSingular}.${OBJECT_EVENT_TRIGGERS[0].value}`,
+          eventName: `${activeObjectMetadataItems[0].nameSingular}.${
+            OBJECT_EVENT_TRIGGERS.find(
+              (availableEvent) => availableEvent.label === name,
+            )?.value
+          }`,
           outputSchema: {},
         },
       };
diff --git a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
index 5c5faa1e4efa..39b76cd7994e 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
@@ -24,7 +24,7 @@ export const getTriggerStepName = (trigger: WorkflowTrigger): string => {
 const getDatabaseEventTriggerStepName = (
   trigger: WorkflowDatabaseEventTrigger,
 ): string => {
-  const [object, action] = trigger.settings.eventName.split('.');
+  const [, action] = trigger.settings.eventName.split('.');
 
-  return `${capitalize(object)} is ${capitalize(action)}`;
+  return `Record is ${capitalize(action)}`;
 };

From a9127462bf0d33d4e2e90e625f13c08ab095f80b Mon Sep 17 00:00:00 2001
From: Thomas Trompette <thomas.trompette@sfr.fr>
Date: Fri, 17 Jan 2025 14:58:58 +0100
Subject: [PATCH 2/4] Add tests

---
 .../getTriggerDefaultDefinition.test.ts       | 129 +++++++++++++-----
 1 file changed, 92 insertions(+), 37 deletions(-)

diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/__tests__/getTriggerDefaultDefinition.test.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/__tests__/getTriggerDefaultDefinition.test.ts
index 8e9244acf906..72459de1b16e 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/__tests__/getTriggerDefaultDefinition.test.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/__tests__/getTriggerDefaultDefinition.test.ts
@@ -1,50 +1,105 @@
+import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
 import { generatedMockObjectMetadataItems } from '~/testing/mock-data/generatedMockObjectMetadataItems';
 import { getTriggerDefaultDefinition } from '../getTriggerDefaultDefinition';
 
-it('throws if the activeObjectMetadataItems list is empty', () => {
-  expect(() => {
-    getTriggerDefaultDefinition({
+describe('getTriggerDefaultDefinition', () => {
+  it('throws if the activeObjectMetadataItems list is empty', () => {
+    expect(() => {
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_CREATED,
+        type: 'DATABASE_EVENT',
+        activeObjectMetadataItems: [],
+      });
+    }).toThrow();
+  });
+
+  it('returns a valid configuration for DATABASE_EVENT trigger type creation', () => {
+    expect(
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_CREATED,
+        type: 'DATABASE_EVENT',
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      }),
+    ).toStrictEqual({
       type: 'DATABASE_EVENT',
-      activeObjectMetadataItems: [],
+      settings: {
+        eventName: `${generatedMockObjectMetadataItems[0].nameSingular}.created`,
+        outputSchema: {},
+      },
     });
-  }).toThrow();
-});
+  });
 
-it('returns a valid configuration for DATABASE_EVENT trigger type', () => {
-  expect(
-    getTriggerDefaultDefinition({
+  it('returns a valid configuration for DATABASE_EVENT trigger type update', () => {
+    expect(
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_UPDATED,
+        type: 'DATABASE_EVENT',
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      }),
+    ).toStrictEqual({
       type: 'DATABASE_EVENT',
-      activeObjectMetadataItems: generatedMockObjectMetadataItems,
-    }),
-  ).toStrictEqual({
-    type: 'DATABASE_EVENT',
-    settings: {
-      eventName: `${generatedMockObjectMetadataItems[0].nameSingular}.created`,
-      outputSchema: {},
-    },
+      settings: {
+        eventName: `${generatedMockObjectMetadataItems[0].nameSingular}.updated`,
+        outputSchema: {},
+      },
+    });
   });
-});
 
-it('returns a valid configuration for MANUAL trigger type', () => {
-  expect(
-    getTriggerDefaultDefinition({
-      type: 'MANUAL',
-      activeObjectMetadataItems: generatedMockObjectMetadataItems,
-    }),
-  ).toStrictEqual({
-    type: 'MANUAL',
-    settings: {
-      objectType: generatedMockObjectMetadataItems[0].nameSingular,
-      outputSchema: {},
-    },
+  it('returns a valid configuration for DATABASE_EVENT trigger type deletion', () => {
+    expect(
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_DELETED,
+        type: 'DATABASE_EVENT',
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      }),
+    ).toStrictEqual({
+      type: 'DATABASE_EVENT',
+      settings: {
+        eventName: `${generatedMockObjectMetadataItems[0].nameSingular}.deleted`,
+        outputSchema: {},
+      },
+    });
   });
-});
 
-it('throws when providing an unknown trigger type', () => {
-  expect(() => {
-    getTriggerDefaultDefinition({
-      type: 'unknown' as any,
-      activeObjectMetadataItems: generatedMockObjectMetadataItems,
+  it('returns a valid configuration for DATABASE_EVENT trigger type creation', () => {
+    expect(
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_CREATED,
+        type: 'DATABASE_EVENT',
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      }),
+    ).toStrictEqual({
+      type: 'DATABASE_EVENT',
+      settings: {
+        eventName: `${generatedMockObjectMetadataItems[0].nameSingular}.created`,
+        outputSchema: {},
+      },
     });
-  }).toThrow('Unknown type: unknown');
+  });
+
+  it('returns a valid configuration for MANUAL trigger type', () => {
+    expect(
+      getTriggerDefaultDefinition({
+        name: 'Launch manually',
+        type: 'MANUAL',
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      }),
+    ).toStrictEqual({
+      type: 'MANUAL',
+      settings: {
+        objectType: generatedMockObjectMetadataItems[0].nameSingular,
+        outputSchema: {},
+      },
+    });
+  });
+
+  it('throws when providing an unknown trigger type', () => {
+    expect(() => {
+      getTriggerDefaultDefinition({
+        name: DatabaseTriggerName.RECORD_IS_CREATED,
+        type: 'unknown' as any,
+        activeObjectMetadataItems: generatedMockObjectMetadataItems,
+      });
+    }).toThrow('Unknown type: unknown');
+  });
 });

From 69d7587c41ac75c30ef88636dd127d572838c9b2 Mon Sep 17 00:00:00 2001
From: Thomas Trompette <thomas.trompette@sfr.fr>
Date: Fri, 17 Jan 2025 15:13:51 +0100
Subject: [PATCH 3/4] Compute name using object event triggers

---
 .../utils/generateWorkflowDiagram.ts                |  7 +++++--
 .../WorkflowEditTriggerDatabaseEventForm.tsx        |  4 ++--
 ...ectEventTriggers.ts => DatabaseTriggerEvents.ts} |  2 +-
 .../utils/getTriggerDefaultDefinition.ts            | 13 ++-----------
 .../workflow-variables/utils/getTriggerStepName.ts  |  5 ++++-
 5 files changed, 14 insertions(+), 17 deletions(-)
 rename packages/twenty-front/src/modules/workflow/workflow-trigger/constants/{ObjectEventTriggers.ts => DatabaseTriggerEvents.ts} (85%)

diff --git a/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts b/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
index 1cdfe811a05f..60d821f49230 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-diagram/utils/generateWorkflowDiagram.ts
@@ -7,9 +7,9 @@ import {
   WorkflowDiagramEdge,
   WorkflowDiagramNode,
 } from '@/workflow/workflow-diagram/types/WorkflowDiagram';
+import { DATABASE_TRIGGER_EVENTS } from '@/workflow/workflow-trigger/constants/DatabaseTriggerEvents';
 
 import { TRIGGER_STEP_ID } from '@/workflow/workflow-trigger/constants/TriggerStepId';
-import { capitalize } from 'twenty-shared';
 import { isDefined } from 'twenty-ui';
 import { v4 } from 'uuid';
 
@@ -70,7 +70,10 @@ export const generateWorkflowDiagram = ({
           trigger.settings.eventName,
         );
 
-        triggerLabel = `Record is ${capitalize(triggerEvent.event)}`;
+        triggerLabel =
+          DATABASE_TRIGGER_EVENTS.find(
+            (event) => event.value === triggerEvent.event,
+          )?.label ?? '';
 
         break;
       }
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
index a84f00bd14ec..cac249cd64dc 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/components/WorkflowEditTriggerDatabaseEventForm.tsx
@@ -4,7 +4,7 @@ import { WorkflowDatabaseEventTrigger } from '@/workflow/types/Workflow';
 import { splitWorkflowTriggerEventName } from '@/workflow/utils/splitWorkflowTriggerEventName';
 import { WorkflowStepBody } from '@/workflow/workflow-steps/components/WorkflowStepBody';
 import { WorkflowStepHeader } from '@/workflow/workflow-steps/components/WorkflowStepHeader';
-import { OBJECT_EVENT_TRIGGERS } from '@/workflow/workflow-trigger/constants/ObjectEventTriggers';
+import { DATABASE_TRIGGER_EVENTS } from '@/workflow/workflow-trigger/constants/DatabaseTriggerEvents';
 import { useTheme } from '@emotion/react';
 import { IconPlaylistAdd, isDefined } from 'twenty-ui';
 
@@ -40,7 +40,7 @@ export const WorkflowEditTriggerDatabaseEventForm = ({
     }));
 
   const selectedEvent = isDefined(triggerEvent)
-    ? OBJECT_EVENT_TRIGGERS.find(
+    ? DATABASE_TRIGGER_EVENTS.find(
         (availableEvent) => availableEvent.value === triggerEvent.event,
       )
     : undefined;
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerEvents.ts
similarity index 85%
rename from packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts
rename to packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerEvents.ts
index 3e7985faa783..4403889e5d44 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/ObjectEventTriggers.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerEvents.ts
@@ -2,7 +2,7 @@ import { SelectOption } from '@/ui/input/components/Select';
 
 import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
 
-export const OBJECT_EVENT_TRIGGERS: Array<SelectOption<string>> = [
+export const DATABASE_TRIGGER_EVENTS: Array<SelectOption<string>> = [
   {
     label: DatabaseTriggerName.RECORD_IS_CREATED,
     value: 'created',
diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
index 78e2af9b9903..04f341145ffa 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/utils/getTriggerDefaultDefinition.ts
@@ -4,14 +4,9 @@ import {
   WorkflowTriggerType,
 } from '@/workflow/types/Workflow';
 import { assertUnreachable } from '@/workflow/utils/assertUnreachable';
-import { DatabaseTriggerName } from '@/workflow/workflow-trigger/constants/DatabaseTriggerName';
-import { OBJECT_EVENT_TRIGGERS } from '@/workflow/workflow-trigger/constants/ObjectEventTriggers';
+import { DATABASE_TRIGGER_EVENTS } from '@/workflow/workflow-trigger/constants/DatabaseTriggerEvents';
 import { getManualTriggerDefaultSettings } from '@/workflow/workflow-trigger/utils/getManualTriggerDefaultSettings';
 
-const isDatabaseName = (name: string): name is DatabaseTriggerName => {
-  return (name as DatabaseTriggerName) !== undefined;
-};
-
 export const getTriggerDefaultDefinition = ({
   name,
   type,
@@ -29,15 +24,11 @@ export const getTriggerDefaultDefinition = ({
 
   switch (type) {
     case 'DATABASE_EVENT': {
-      if (!isDatabaseName(name)) {
-        throw new Error('Database event has an invalid name');
-      }
-
       return {
         type,
         settings: {
           eventName: `${activeObjectMetadataItems[0].nameSingular}.${
-            OBJECT_EVENT_TRIGGERS.find(
+            DATABASE_TRIGGER_EVENTS.find(
               (availableEvent) => availableEvent.label === name,
             )?.value
           }`,
diff --git a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
index 39b76cd7994e..00fb841b792c 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/getTriggerStepName.ts
@@ -3,6 +3,7 @@ import {
   WorkflowTrigger,
 } from '@/workflow/types/Workflow';
 import { assertUnreachable } from '@/workflow/utils/assertUnreachable';
+import { DATABASE_TRIGGER_EVENTS } from '@/workflow/workflow-trigger/constants/DatabaseTriggerEvents';
 import { capitalize } from 'twenty-shared';
 import { isDefined } from 'twenty-ui';
 
@@ -26,5 +27,7 @@ const getDatabaseEventTriggerStepName = (
 ): string => {
   const [, action] = trigger.settings.eventName.split('.');
 
-  return `Record is ${capitalize(action)}`;
+  return (
+    DATABASE_TRIGGER_EVENTS.find((event) => event.value === action)?.label ?? ''
+  );
 };

From 7ef16b2659a51ce8470cb3baa104f39c08da0984 Mon Sep 17 00:00:00 2001
From: Thomas Trompette <thomas.trompette@sfr.fr>
Date: Fri, 17 Jan 2025 16:20:22 +0100
Subject: [PATCH 4/4] Fix icons and test

---
 .../workflow-trigger/constants/DatabaseTriggerTypes.ts        | 4 ++--
 .../utils/__tests__/getTriggerStepName.test.ts                | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
index fef99520cc1d..9fd490531111 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-trigger/constants/DatabaseTriggerTypes.ts
@@ -15,11 +15,11 @@ export const DATABASE_TRIGGER_TYPES: Array<{
   {
     name: DatabaseTriggerName.RECORD_IS_UPDATED,
     type: 'DATABASE_EVENT',
-    icon: IconTrash,
+    icon: IconClick,
   },
   {
     name: DatabaseTriggerName.RECORD_IS_DELETED,
     type: 'DATABASE_EVENT',
-    icon: IconClick,
+    icon: IconTrash,
   },
 ];
diff --git a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/__tests__/getTriggerStepName.test.ts b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/__tests__/getTriggerStepName.test.ts
index f5cfa268054f..377d182f48a1 100644
--- a/packages/twenty-front/src/modules/workflow/workflow-variables/utils/__tests__/getTriggerStepName.test.ts
+++ b/packages/twenty-front/src/modules/workflow/workflow-variables/utils/__tests__/getTriggerStepName.test.ts
@@ -10,7 +10,7 @@ it('returns the expected name for a DATABASE_EVENT trigger', () => {
         outputSchema: {},
       },
     }),
-  ).toBe('Company is Created');
+  ).toBe('Record is Created');
 });
 
 it('returns the expected name for a MANUAL trigger without a defined objectType', () => {