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

package/package.json

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

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 {