@capillarytech/creatives-library 8.0.139-alpha.2 → 8.0.139

package/package.json

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

package/v2Containers/MobilePushNew/components/MediaUploaders.js

@@ -138,7 +138,7 @@ const MediaUploaders = ({
   // Move updateCarouselCard to top level so useEffect can access it
   const updateCarouselCard = useCallback((index, fields) => {
     const currentData = getCurrentCarouselData();
-    setCurrentCarouselData(currentData.map((card, i) => (
+    setCurrentCarouselData(currentData?.map((card, i) => (
       i === index ? { ...card, ...fields } : card
     )));
   }, [getCurrentCarouselData, setCurrentCarouselData]);
@@ -513,10 +513,9 @@ const MediaUploaders = ({
           imageSrc={imageSource}
           updateImageSrc={(src) => {
             // For gallery uploads (direct URL), update immediately  
-            if (src && (src.includes('http') || src.includes('https'))) {
+            if (src && src?.includes('http')) {
               handleCarouselImageChange(src, cardIndex);
-            }
-            
+            }          
             // Always call setUpdateMpushImageSrc for Redux state management
             setUpdateMpushImageSrc(src, activeTab === ANDROID ? 0 : 1, CAROUSEL);
           }}
@@ -849,14 +848,14 @@ const MediaUploaders = ({
       const platformIndex = activeTab === ANDROID ? 0 : 1;
       const uploadedData = imageData[`uploadedAssetData${platformIndex}`];
       
-      if (uploadedData && uploadedData.metaInfo && uploadedData.metaInfo.secure_file_path) {
+      if (uploadedData?.metaInfo?.secure_file_path) {
         // Check if this is new upload data (different from what we processed last time)
         const currentDataKey = `${uploadedData.metaInfo.secure_file_path}_${uploadedData.metaInfo.file_name || ''}`;
         const previousDataKey = previousUploadDataRef.current;
         
         if (currentDataKey !== previousDataKey) {
           // Update the specific card that was uploading
-          updateCarouselCard(uploadingCardIndex, { imageUrl: uploadedData.metaInfo.secure_file_path });
+          updateCarouselCard(uploadingCardIndex, { imageUrl: uploadedData?.metaInfo?.secure_file_path });
           
           // Store this upload data as "processed" to avoid reprocessing
           previousUploadDataRef.current = currentDataKey;

package/v2Containers/MobilePushNew/index.js

@@ -152,7 +152,7 @@ const processMediaType = (contentType, expandableDetails) => {
   if (expandableStyle === BIG_PICTURE) {
     mediaType = IMAGE;
     imageSrc = expandableDetails?.image || contentType?.image;
-  } else if (expandableStyle === MANUAL_CAROUSEL || expandableStyle === AUTO_CAROUSEL || expandableStyle === FILMSTRIP_CAROUSEL || expandableStyle === CAROUSEL) {
+  } else if ([MANUAL_CAROUSEL, AUTO_CAROUSEL, FILMSTRIP_CAROUSEL, CAROUSEL].includes(expandableStyle)) {
     mediaType = CAROUSEL;
     // Extract carousel data from expandableDetails
     carouselData = expandableCarouselData || [];
@@ -165,7 +165,7 @@ const processMediaType = (contentType, expandableDetails) => {
   if (expandableMedia?.length > 0) {
     const mediaItem = expandableMedia[0];
     const { type, url, videoPreviewUrl } = mediaItem || {};
-    
+
     if (type === VIDEO) {
       // Enhanced GIF detection with robust URL validation
       const isGif = (() => {
@@ -174,7 +174,7 @@ const processMediaType = (contentType, expandableDetails) => {
         if (!trimmedUrl) return false;
         return trimmedUrl.toLowerCase().includes('.gif');
       })();
-      
+
       if (isGif) {
         mediaType = GIF;
       } else {
@@ -191,7 +191,7 @@ const processMediaType = (contentType, expandableDetails) => {
 
   // PRIORITY 3: Check mediaTypeFromStyle (from expandable details)
   // This provides media type information from the expandable details structure
-  if (mediaTypeFromStyle === VIDEO || mediaTypeFromStyle === GIF || mediaTypeFromStyle === CAROUSEL || mediaTypeFromStyle === IMAGE) {
+  if ([VIDEO, GIF, CAROUSEL, IMAGE].includes(mediaTypeFromStyle)) {
     mediaType = mediaTypeFromStyle;
     // If mediaType from style is CAROUSEL, also check for carousel data
     if (mediaTypeFromStyle === CAROUSEL && expandableCarouselData?.length > 0) {
@@ -201,7 +201,7 @@ const processMediaType = (contentType, expandableDetails) => {
 
   // PRIORITY 4: Check mediaTypeFromRoot (from root content type)
   // This provides media type information from the root content structure
-  if (mediaTypeFromRoot === VIDEO || mediaTypeFromRoot === GIF || mediaTypeFromRoot === CAROUSEL || mediaTypeFromRoot === IMAGE) {
+  if ([VIDEO, GIF, CAROUSEL, IMAGE].includes(mediaTypeFromRoot)) {
     mediaType = mediaTypeFromRoot;
     // If mediaType from root is CAROUSEL, also check for carousel data
     if (mediaTypeFromRoot === CAROUSEL && expandableCarouselData?.length > 0) {
@@ -211,7 +211,7 @@ const processMediaType = (contentType, expandableDetails) => {
 
   // PRIORITY 5: Check rootType (lowest priority)
   // This is the basic type from the root content type
-  if (rootType === VIDEO || rootType === GIF || rootType === CAROUSEL) {
+  if ([VIDEO, GIF, CAROUSEL].includes(rootType)) {
     mediaType = rootType;
     // If root type is CAROUSEL, also check for carousel data
     if (rootType === CAROUSEL && expandableCarouselData?.length > 0) {
@@ -301,7 +301,7 @@ const createContentData = (contentType, mediaInfo, buttons, deepLink, processCar
     // Otherwise, check carousel data
     if (mediaType === CAROUSEL) {
       const carouselLinkInfo = processCarouselDataForDeepLinks(carouselData);
-      return carouselLinkInfo.deepLinkValue || carouselLinkInfo.externalLinkValue;
+      return carouselLinkInfo?.deepLinkValue || carouselLinkInfo?.externalLinkValue;
     }
     return false;
   })();
@@ -321,7 +321,7 @@ const createContentData = (contentType, mediaInfo, buttons, deepLink, processCar
     }
     if (mediaType === CAROUSEL) {
       const carouselLinkInfo = processCarouselDataForDeepLinks(carouselData);
-      return carouselLinkInfo.externalLinkValue ? EXTERNAL_LINK : DEEP_LINK;
+      return carouselLinkInfo?.externalLinkValue ? EXTERNAL_LINK : DEEP_LINK;
     }
     return DEEP_LINK;
   })();
@@ -400,7 +400,7 @@ const createContentData = (contentType, mediaInfo, buttons, deepLink, processCar
       }
       if (mediaType === CAROUSEL) {
         const carouselLinkInfo = processCarouselDataForDeepLinks(carouselData);
-        return carouselLinkInfo.deepLinkValue;
+        return carouselLinkInfo?.deepLinkValue;
       }
       return "";
     })(),
@@ -426,7 +426,7 @@ const createContentData = (contentType, mediaInfo, buttons, deepLink, processCar
       }
       if (mediaType === CAROUSEL) {
         const carouselLinkInfo = processCarouselDataForDeepLinks(carouselData);
-        return carouselLinkInfo.deepLinkKeysValue;
+        return carouselLinkInfo?.deepLinkKeysValue;
       }
       return [];
     })(),
@@ -449,7 +449,7 @@ const createContentData = (contentType, mediaInfo, buttons, deepLink, processCar
       }
       if (mediaType === CAROUSEL) {
         const carouselLinkInfo = processCarouselDataForDeepLinks(carouselData);
-        return carouselLinkInfo.externalLinkValue;
+        return carouselLinkInfo?.externalLinkValue;
       }
       return "";
     })(),
@@ -1445,13 +1445,13 @@ const MobilePushNew = ({
       }
 
       // Check all possible media type sources
-      if (iosType === VIDEO || iosType === GIF || iosType === CAROUSEL) {
+      if ([VIDEO, GIF, CAROUSEL].includes(iosType)) {
         iosMediaType = iosType;
       }
-      if (iosMediaTypeFromRoot === VIDEO || iosMediaTypeFromRoot === GIF || iosMediaTypeFromRoot === CAROUSEL || iosMediaTypeFromRoot === IMAGE) {
+      if ([VIDEO, GIF, CAROUSEL, IMAGE].includes(iosMediaTypeFromRoot)) {
         iosMediaType = iosMediaTypeFromRoot;
       }
-      if (iosMediaTypeFromStyle === VIDEO || iosMediaTypeFromStyle === GIF || iosMediaTypeFromStyle === CAROUSEL || iosMediaTypeFromStyle === IMAGE) {
+      if ([VIDEO, GIF, CAROUSEL, IMAGE].includes(iosMediaTypeFromStyle)) {
         iosMediaType = iosMediaTypeFromStyle;
       }