fix: remove unsupported types (#726)

Author: CarolinaMoraes

src/common/interfaces/email-api-options.interface.ts

@@ -25,6 +25,6 @@ export interface EmailApiOptions {
   attachments?: EmailApiAttachment[];
   template?: {
     id: string;
-    variables?: Record<string, string | number | boolean>;
+    variables?: Record<string, string | number>;
   };
 }

src/common/utils/parse-template-to-api-options.spec.ts

@@ -38,11 +38,6 @@ describe('parseTemplateToApiOptions', () => {
           fallbackValue: 'Subscriber',
           type: 'string',
         },
-        {
-          key: 'isVip',
-          fallbackValue: false,
-          type: 'boolean',
-        },
       ],
     };
 
@@ -62,11 +57,6 @@ describe('parseTemplateToApiOptions', () => {
           fallback_value: 'Subscriber',
           type: 'string',
         },
-        {
-          key: 'isVip',
-          fallback_value: false,
-          type: 'boolean',
-        },
       ],
     });
   });
@@ -158,11 +148,6 @@ describe('parseTemplateToApiOptions', () => {
           fallbackValue: 42,
           type: 'number',
         },
-        {
-          key: 'isEnabled',
-          fallbackValue: true,
-          type: 'boolean',
-        },
         {
           key: 'optional',
           fallbackValue: null,
@@ -184,11 +169,6 @@ describe('parseTemplateToApiOptions', () => {
         fallback_value: 42,
         type: 'number',
       },
-      {
-        key: 'isEnabled',
-        fallback_value: true,
-        type: 'boolean',
-      },
       {
         key: 'optional',
         fallback_value: null,
@@ -219,131 +199,4 @@ describe('parseTemplateToApiOptions', () => {
 
     expect(apiOptions.variables).toEqual([]);
   });
-
-  it('handles object and list variable types for create template', () => {
-    const templatePayload: CreateTemplateOptions = {
-      name: 'Complex Variables Template',
-      html: '<h1>Complex template</h1>',
-      variables: [
-        {
-          key: 'userProfile',
-          type: 'object',
-          fallbackValue: { name: 'John', age: 30 },
-        },
-        {
-          key: 'tags',
-          type: 'list',
-          fallbackValue: ['premium', 'vip'],
-        },
-        {
-          key: 'scores',
-          type: 'list',
-          fallbackValue: [95, 87, 92],
-        },
-        {
-          key: 'flags',
-          type: 'list',
-          fallbackValue: [true, false, true],
-        },
-        {
-          key: 'items',
-          type: 'list',
-          fallbackValue: [{ id: 1 }, { id: 2 }],
-        },
-      ],
-    };
-
-    const apiOptions = parseTemplateToApiOptions(templatePayload);
-
-    expect(apiOptions.variables).toEqual([
-      {
-        key: 'userProfile',
-        type: 'object',
-        fallback_value: { name: 'John', age: 30 },
-      },
-      {
-        key: 'tags',
-        type: 'list',
-        fallback_value: ['premium', 'vip'],
-      },
-      {
-        key: 'scores',
-        type: 'list',
-        fallback_value: [95, 87, 92],
-      },
-      {
-        key: 'flags',
-        type: 'list',
-        fallback_value: [true, false, true],
-      },
-      {
-        key: 'items',
-        type: 'list',
-        fallback_value: [{ id: 1 }, { id: 2 }],
-      },
-    ]);
-  });
-
-  it('handles object and list variable types for update template', () => {
-    const updatePayload: UpdateTemplateOptions = {
-      subject: 'Updated Complex Template',
-      variables: [
-        {
-          key: 'config',
-          type: 'object',
-          fallbackValue: { theme: 'dark', lang: 'en' },
-        },
-        {
-          key: 'permissions',
-          type: 'list',
-          fallbackValue: ['read', 'write'],
-        },
-        {
-          key: 'counts',
-          type: 'list',
-          fallbackValue: [10, 20, 30],
-        },
-        {
-          key: 'enabled',
-          type: 'list',
-          fallbackValue: [true, false],
-        },
-        {
-          key: 'metadata',
-          type: 'list',
-          fallbackValue: [{ key: 'a' }, { key: 'b' }],
-        },
-      ],
-    };
-
-    const apiOptions = parseTemplateToApiOptions(updatePayload);
-
-    expect(apiOptions.variables).toEqual([
-      {
-        key: 'config',
-        type: 'object',
-        fallback_value: { theme: 'dark', lang: 'en' },
-      },
-      {
-        key: 'permissions',
-        type: 'list',
-        fallback_value: ['read', 'write'],
-      },
-      {
-        key: 'counts',
-        type: 'list',
-        fallback_value: [10, 20, 30],
-      },
-      {
-        key: 'enabled',
-        type: 'list',
-        fallback_value: [true, false],
-      },
-      {
-        key: 'metadata',
-        type: 'list',
-        fallback_value: [{ key: 'a' }, { key: 'b' }],
-      },
-    ]);
-  });
 });

src/common/utils/parse-template-to-api-options.ts

@@ -1,17 +1,10 @@
 import type { CreateTemplateOptions } from '../../templates/interfaces/create-template-options.interface';
-import type { TemplateVariableListFallbackType } from '../../templates/interfaces/template';
 import type { UpdateTemplateOptions } from '../../templates/interfaces/update-template.interface';
 
 interface TemplateVariableApiOptions {
   key: string;
-  type: 'string' | 'number' | 'boolean' | 'object' | 'list';
-  fallback_value?:
-    | string
-    | number
-    | boolean
-    | Record<string, unknown>
-    | TemplateVariableListFallbackType
-    | null;
+  type: 'string' | 'number';
+  fallback_value?: string | number | null;
 }
 
 interface TemplateApiOptions {

src/emails/interfaces/create-email-options.interface.ts

@@ -28,7 +28,7 @@ interface EmailRenderOptions {
 interface EmailTemplateOptions {
   template: {
     id: string;
-    variables?: Record<string, string | number | boolean>;
+    variables?: Record<string, string | number>;
   };
 }