@capillarytech/creatives-library 8.0.152 → 8.0.153-beta.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@capillarytech/creatives-library",
   "author": "meharaj",
-  "version": "8.0.152",
+  "version": "8.0.153-beta.1",
   "description": "Capillary creatives ui",
   "main": "./index.js",
   "module": "./index.es.js",

package/v2Containers/CreativesContainer/SlideBoxFooter.js

@@ -33,6 +33,22 @@ function SlideBoxFooter(props) {
     shouldShowContinueFooter,
   } = props;
 
+  console.log("SlideBoxFooter", {
+    slidBoxContent,
+    onSave,
+    onEditTemplate,
+    onCreateNextStep,
+    isFullMode,
+    fetchingCmsData,
+    isTemplateNameEmpty,
+    errorMessages,
+    isLiquidValidationError,
+    currentTab,
+    onTestAndPreview,
+    showTestAndPreviewButton,
+    shouldShowContinueFooter: shouldShowContinueFooter(),
+    shouldShowDoneFooter: shouldShowDoneFooter(),
+  });
   return (
     <div className='template-footer-width'>
       {isLiquidValidationError && (<ErrorInfoNote errorMessages={errorMessages} currentTab={currentTab?.toUpperCase()} />)}

package/v2Containers/CreativesContainer/index.js

@@ -200,12 +200,19 @@ export class Creatives extends React.Component {
   }
 
   onCreateNextStep = () => {
+    console.log("onCreateNextStep 1");
     this.setState((prevState) => {
       let templateStep = prevState.templateStep + 1;
       const { emailCreateMode, currentChannel } = prevState;
       if ((currentChannel.toUpperCase() === constants.EMAIL && emailCreateMode === "upload") || [constants.MOBILE_PUSH, constants.WECHAT, constants.INAPP].includes(currentChannel.toUpperCase())) {
         templateStep = prevState.templateStep + 2;
       }
+      console.log("onCreateNextStep 2", {
+        templateStep,
+        currentChannel,
+        emailCreateMode,
+        prevState,
+      });
       return {
         templateStep,
       };
@@ -1353,7 +1360,17 @@ export class Creatives extends React.Component {
       }
     }
 
-
+    console.log("shouldShowDoneFooter", {
+      showDone,
+      channel,
+      currentStep,
+      slidBoxContent,
+      templateStep,
+      currentChannel,
+      templateData,
+      isFullMode,
+      isEmail: channel === constants.EMAIL,
+    });
     return showDone;
   }
 
@@ -1432,7 +1449,19 @@ export class Creatives extends React.Component {
     } else if (currentChannel.toUpperCase() === constants.INAPP) {
       isShowContinueFooter = !isEmpty(inAppCreateMode) && currentChannel === "modeSelection";
     }
-
+    console.log("shouldShowContinueFooter", {
+      isShowContinueFooter,
+      mobilePushCreateMode,
+      weChatTemplateType,
+      inAppCreateMode,
+      currentStep,
+      currentChannel,
+      slidBoxContent,
+      templateStep,
+      emailCreateMode,
+      emailLayout: this.props?.EmailLayout,
+      isEmail: channel === constants.EMAIL,
+    });
     return isShowContinueFooter;
   }
 

package/v2Containers/EmailWrapper/hooks/useEmailWrapper.js

@@ -1,4 +1,4 @@
-import { useState, useEffect, useMemo, useCallback } from 'react';
+import { useState, useEffect, useMemo, useCallback, useRef } from 'react';
 import isEmpty from 'lodash/isEmpty';
 import get from 'lodash/get';
 import find from 'lodash/find';
@@ -66,6 +66,9 @@ const useEmailWrapper = ({
     query: { module: 'library', type: 'embedded' },
   });
 
+  // Prevent duplicate uploads from rapid/double onChange events
+  const isUploadingRef = useRef(false);
+
   // Cleanup effect
   useEffect(() => {
     return () => {
@@ -114,6 +117,11 @@ const useEmailWrapper = ({
   }, [formatMessage]);
 
   const handleFileUpload = useCallback((file = {}) => {
+    // Guard against duplicate triggers (e.g., multiple onChange emissions)
+    if (isUploadingRef.current || isUploading) {
+      return;
+    }
+
     if (!isUploading) {
       // Validate file object
       if (!file || !file.name) {
@@ -147,11 +155,20 @@ const useEmailWrapper = ({
           return;
         }
 
+        // Mark uploading to prevent duplicate dispatches before Redux state updates
+        isUploadingRef.current = true;
+
         templatesActions.handleZipUpload(file.originFileObj, () => { // Handle upload success
           stopTimerGA();
           setModeContent({ file });
+          // Clear local uploading guard before navigating to next step
+          isUploadingRef.current = false;
           showNextStep();
-        }, handleZipUploadError);
+        }, () => {
+          // Clear local uploading guard on error as well
+          isUploadingRef.current = false;
+          handleZipUploadError();
+        });
       } else if (fileExtension === 'html' || fileExtension === 'htm') {
         // Verify originFileObj exists
         if (!file.originFileObj) {
@@ -159,12 +176,16 @@ const useEmailWrapper = ({
           return;
         }
 
+        // Guard during HTML processing to avoid duplicate reads
+        isUploadingRef.current = true;
+
         const reader = new FileReader();
         reader.onload = () => {
           const text = reader.result;
           // Add defensive check for empty or invalid content
           if (!text || typeof text !== 'string') {
             handleZipUploadError();
+            isUploadingRef.current = false;
             return;
           }
           
@@ -173,9 +194,12 @@ const useEmailWrapper = ({
           
           setModeContent({ file });
           templatesActions.handleHtmlUpload(text);
+          // Clear uploading guard after handing off HTML upload
+          isUploadingRef.current = false;
         };
         reader.onerror = () => {
           handleZipUploadError();
+          isUploadingRef.current = false;
         };
         reader.readAsText(file.originFileObj);
       } else {