@capillarytech/creatives-library 8.0.292-alpha.0 → 8.0.293

package/v2Containers/SmsWrapper/index.js

@@ -36,7 +36,6 @@ const SmsWrapper = (props) => {
     handleTestAndPreview,
     handleCloseTestAndPreview,
     isTestAndPreviewMode,
-    onValidationFail,
     } = props;
 
   const smsProps = {
@@ -59,7 +58,6 @@ const SmsWrapper = (props) => {
     handleTestAndPreview,
     handleCloseTestAndPreview,
     isTestAndPreviewMode,
-    onValidationFail,
   };
   const isTraiDlt = isTraiDLTEnable(isFullMode, smsRegister);
   return <>

package/v2Containers/Viber/index.js

@@ -234,6 +234,7 @@ export const Viber = (props) => {
     const { valid, isBraceError } = validateTags({
       content: value,
       tagsParam: tags,
+      injectedTagsParams: injectedTags,
       location,
       tagModule: 'outbound',
       isFullMode,

package/v2Containers/WebPush/Create/hooks/useTagManagement.js

@@ -105,10 +105,12 @@ export const useTagManagement = ({
   const validationConfig = useMemo(
     () => ({
       tagsParam: tags,
+      injectedTagsParams: injectedTags,
       location,
       tagModule: getDefaultTags,
+      eventContextTags,
     }),
-    [tags, location, getDefaultTags],
+    [tags, injectedTags, location, getDefaultTags, eventContextTags],
   );
 
   return {

package/v2Containers/WebPush/Create/hooks/useTagManagement.test.js

@@ -528,19 +528,26 @@ describe('useTagManagement', () => {
 
   describe('validationConfig', () => {
     it('should return validation config with tags', () => {
+      const injectedTags = [{ id: 5, name: 'Injected Tag' }];
+      const eventContextTags = [{ id: 6, name: 'Event Tag' }];
+
       const { result } = renderHook(() =>
         useTagManagement({
           location: defaultLocation,
           globalActions: mockGlobalActions,
           metaEntities: defaultMetaEntities,
+          injectedTags,
+          eventContextTags,
           getDefaultTags: 'custom',
         })
       );
 
       expect(result.current.validationConfig).toEqual({
         tagsParam: defaultMetaEntities.tags.standard,
+        injectedTagsParams: injectedTags,
         location: defaultLocation,
         tagModule: 'custom',
+        eventContextTags,
       });
     });
 

package/v2Containers/WebPush/Create/index.js

@@ -295,8 +295,8 @@ const WebPushCreate = ({
   const validateTemplateName = useCallback((value) => validateTemplateNameUtil(value), []);
 
   const validateTitle = useCallback(
-    (value) => validateTitleUtil(value, formatMessage, messages, restrictPersonalization, validationConfig, isFullMode),
-    [formatMessage, restrictPersonalization, validationConfig, isFullMode],
+    (value) => validateTitleUtil(value, formatMessage, messages, restrictPersonalization),
+    [formatMessage, restrictPersonalization],
   );
 
   const validateUrl = useCallback(

package/v2Containers/WebPush/Create/utils/validation.js

@@ -11,34 +11,19 @@ import { hasPersonalizationTags } from '../../../../utils/commonUtils';
 export const validateTemplateName = (value) => !value || value.trim() === '';
 
 /**
- * Validates notification title (required and optional tag validation)
+ * Validates notification title
  * @param {string} value - The title value
  * @param {Function} formatMessage - i18n format message function
  * @param {Object} messages - Message definitions
- * @param {Object} [validationConfig] - Optional config for tag validation
- * @param {boolean} [isFullMode] - Optional; when set with validationConfig, runs tag validation
  * @returns {string} Error message if invalid, empty string if valid
  */
-export const validateTitle = (value, formatMessage, messages, restrictPersonalization, validationConfig, isFullMode) => {
+export const validateTitle = (value, formatMessage, messages, restrictPersonalization) => {
   if (!value || value.trim() === '') {
     return formatMessage(messages.titleRequired);
   }
   if (restrictPersonalization && hasPersonalizationTags(value)) {
     return formatMessage(messages.personalizationTokensErrorMessage);
   }
-
-  if (validationConfig != null) {
-    const validationResponse = validateTags({
-      content: value,
-      ...validationConfig,
-      isFullMode,
-    }) || {};
-
-    if (validationResponse?.isBraceError) {
-      return formatMessage(globalMessages.unbalanacedCurlyBraces);
-    }
-  }
-
   return '';
 };
 

package/v2Containers/WebPush/Create/utils/validation.test.js

@@ -127,40 +127,6 @@ describe('validation', () => {
       expect(result).toBe('Personalization tags are not supported for anonymous customers');
       expect(mockFormatMessage).toHaveBeenCalledWith(mockMessages.personalizationTokensErrorMessage);
     });
-
-    it('should return brace error when validationConfig is provided and validateTags returns isBraceError', () => {
-      const validationConfig = { tagsParam: [], location: {}, tagModule: '' };
-      validateTags.mockReturnValue({ isBraceError: true });
-      const result = validateTitle(
-        'Valid Title',
-        mockFormatMessage,
-        mockMessages,
-        false,
-        validationConfig,
-        false
-      );
-      expect(result).toBe('Unbalanced curly braces');
-      expect(mockFormatMessage).toHaveBeenCalledWith(globalMessages.unbalanacedCurlyBraces);
-      expect(validateTags).toHaveBeenCalledWith({
-        content: 'Valid Title',
-        ...validationConfig,
-        isFullMode: false,
-      });
-    });
-
-    it('should not run tag validation when validationConfig is null', () => {
-      validateTags.mockClear();
-      const result = validateTitle('Valid Title', mockFormatMessage, mockMessages, false, null);
-      expect(result).toBe('');
-      expect(validateTags).not.toHaveBeenCalled();
-    });
-
-    it('should not run tag validation when validationConfig is undefined', () => {
-      validateTags.mockClear();
-      const result = validateTitle('Valid Title', mockFormatMessage, mockMessages, false, undefined);
-      expect(result).toBe('');
-      expect(validateTags).not.toHaveBeenCalled();
-    });
   });
 
   describe('validateUrl', () => {
@@ -217,8 +183,10 @@ describe('validation', () => {
   describe('validateMessageContent', () => {
     const mockValidationConfig = {
       tagsParam: [],
+      injectedTagsParams: [],
       location: {},
       tagModule: '',
+      eventContextTags: [],
     };
 
     beforeEach(() => {
@@ -257,6 +225,17 @@ describe('validation', () => {
       });
     });
 
+    it('should return error message for unsupported tags', () => {
+      validateTags.mockReturnValue({
+        unsupportedTags: ['tag1', 'tag2'],
+      });
+      const result = validateMessageContent('Message with {tag1} and {tag2}', mockFormatMessage, mockMessages, mockValidationConfig);
+      expect(result).toBe('Unsupported tags: tag1, tag2');
+      expect(mockFormatMessage).toHaveBeenCalledWith(globalMessages.unsupportedTagsValidationError, {
+        unsupportedTags: 'tag1, tag2',
+      });
+    });
+
     it('should return error message for unbalanced curly braces', () => {
       validateTags.mockReturnValue({
         isBraceError: true,
@@ -266,11 +245,22 @@ describe('validation', () => {
       expect(mockFormatMessage).toHaveBeenCalledWith(globalMessages.unbalanacedCurlyBraces);
     });
 
+    it('should return error message for both unsupported tags and brace error (unsupported tags takes precedence)', () => {
+      validateTags.mockReturnValue({
+        unsupportedTags: ['tag1'],
+        isBraceError: true,
+      });
+      const result = validateMessageContent('Message with {tag1}', mockFormatMessage, mockMessages, mockValidationConfig);
+      expect(result).toBe('Unsupported tags: tag1');
+    });
+
     it('should pass validation config to validateTags', () => {
       const customConfig = {
         tagsParam: [{ id: 1, name: 'Tag1' }],
+        injectedTagsParams: [{ id: 2, name: 'Tag2' }],
         location: { query: { type: 'test' } },
         tagModule: 'custom',
+        eventContextTags: [{ id: 3, name: 'Tag3' }],
       };
       validateTags.mockReturnValue({});
       validateMessageContent('Valid message', mockFormatMessage, mockMessages, customConfig);
@@ -316,31 +306,6 @@ describe('validation', () => {
       expect(result).toBe('Personalization tags are not supported for anonymous customers');
       expect(mockFormatMessage).toHaveBeenCalledWith(mockMessages.personalizationTokensErrorMessage);
     });
-
-    it('should return unsupported tags error when validateTags returns unsupportedTags', () => {
-      validateTags.mockReturnValue({ unsupportedTags: ['invalidTag', 'otherTag'] });
-      const result = validateMessageContent(
-        'Hello {{invalidTag}}',
-        mockFormatMessage,
-        mockMessages,
-        mockValidationConfig
-      );
-      expect(result).toBe('Unsupported tags: invalidTag, otherTag');
-      expect(mockFormatMessage).toHaveBeenCalledWith(
-        globalMessages.unsupportedTagsValidationError,
-        { unsupportedTags: 'invalidTag, otherTag' }
-      );
-    });
-
-    it('should pass isFullMode to validateTags when provided', () => {
-      validateTags.mockReturnValue({});
-      validateMessageContent('Valid message', mockFormatMessage, mockMessages, mockValidationConfig, true);
-      expect(validateTags).toHaveBeenCalledWith({
-        content: 'Valid message',
-        ...mockValidationConfig,
-        isFullMode: true,
-      });
-    });
   });
 });
 

package/v2Containers/Whatsapp/index.js

@@ -637,21 +637,22 @@ export const Whatsapp = (props) => {
         validateTags({
           content: contentData.join(""),
           tagsParam: tags,
+          injectedTagsParams: injectedTags,
           location,
           tagModule: getDefaultTags,
+          eventContextTags,
           isFullMode,
         }) || {};
+      const unsupportedTagsLengthCheck =
+        validationResponse?.unsupportedTags?.length > 0;
       if (type === HEADER_TEXT) {
         headerTagValidationResponse = validationResponse;
-        updateIsHeaderTagValidationError(validationResponse.isBraceError);
+        updateIsHeaderTagValidationError(unsupportedTagsLengthCheck);
       } else if (type === CAROUSEL_TEXT) {
-        return [
-          { fieldName: "carouselTagValidationErrMessage", value: validationResponse.isBraceError ? validationResponse : {} },
-          { fieldName: "carouselTagValidationErr", value: validationResponse.isBraceError },
-        ];
+        return [{fieldName: "carouselTagValidationErrMessage", value: validationResponse}, {fieldName: "carouselTagValidationErr", value: unsupportedTagsLengthCheck}];
       } else {
         tagValidationResponse = validationResponse;
-        updateIsTagValidationError(validationResponse.isBraceError);
+        updateIsTagValidationError(unsupportedTagsLengthCheck);
       }
     }
   };
@@ -2681,11 +2682,18 @@ const isAuthenticationTemplate = isEqual(templateCategory, WHATSAPP_CATEGORIES.a
     } else {
       validationResponse = tagValidationResponse;
     }
-    const { isBraceError } = validationResponse || {};
+    const { unsupportedTags = [], isBraceError } = validationResponse;
+
+    let tagError = "";
+    if (unsupportedTags.length > 0) {
+      tagError = formatMessage(globalMessages.unsupportedTagsValidationError, {
+        unsupportedTags,
+      });
+    }
     if (isBraceError) {
-      return <CapError>{formatMessage(globalMessages.unbalanacedCurlyBraces)}</CapError>;
+      tagError = formatMessage(globalMessages.unbalanacedCurlyBraces);
     }
-    return null;
+    return <CapError>{tagError}</CapError>;
   };
 
   const editModeContent = () => (
@@ -3204,7 +3212,7 @@ const isAuthenticationTemplate = isEqual(templateCategory, WHATSAPP_CATEGORIES.a
       <TestAndPreviewSlidebox
         show={propsShowTestAndPreviewSlidebox || showTestAndPreviewSlidebox}
         onClose={handleCloseTestAndPreview}
-        formData={null} // WhatsApp doesn't use formData structure like SMS
+        formData={getTemplateContent()}
         content={getTemplateContent()}
         currentChannel={WHATSAPP}
       />