@capillarytech/creatives-library 9.0.18 → 9.0.21

package/package.json

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

package/v2Containers/EmailWrapper/hooks/useEmailWrapper.js

@@ -234,13 +234,18 @@ const useEmailWrapper = ({
     // New flow: When template details are loaded and it's a BEE template, set it in Templates.BEETemplate
     // This allows Email component to properly initialize and call getCmsSetting
     const hasTemplateDetails = Email?.templateDetails && !isEmpty(Email.templateDetails);
+    // An inline templateData prop (e.g. library add-creative -> select template)
+    // is authoritative when template details weren't fetched (the by-id fetch is
+    // skipped whenever a templateData prop is present). Without seeding from it,
+    // setBEETemplate never fires for that flow and a BEE template opens as HTML.
+    const hasTemplateDataProp = templateData && !isEmpty(templateData);
     // Note: We check Email?.BEETemplate as a proxy, but the actual BEETemplate is in Templates reducer
     // The Email component will detect it via this.props.Templates.BEETemplate
     const hasBEETemplate = Email?.BEETemplate && !isEmpty(Email.BEETemplate);
 
-    if (hasTemplateDetails && !hasBEETemplate && templatesActions?.setBEETemplate && !beeTemplateSetRef.current) {
+    if ((hasTemplateDetails || hasTemplateDataProp) && !hasBEETemplate && templatesActions?.setBEETemplate && !beeTemplateSetRef.current) {
       // Check if it's a BEE template
-      const editTemplateData = Email.templateDetails;
+      const editTemplateData = hasTemplateDetails ? Email.templateDetails : templateData;
       const getIsDragDrop = (data) => {
         if (!data) return false;
         const baseDragDrop = get(data, 'versions.base.is_drag_drop', false);
@@ -295,11 +300,11 @@ const useEmailWrapper = ({
       }
     }
 
-    // Reset ref when template changes (template details cleared)
-    if (!hasTemplateDetails && beeTemplateSetRef.current) {
+    // Reset ref when template changes (template details cleared and no inline prop)
+    if (!hasTemplateDetails && !hasTemplateDataProp && beeTemplateSetRef.current) {
       beeTemplateSetRef.current = false;
     }
-  }, [Email?.templateDetails, Email?.BEETemplate, Email?.getTemplateDetailsInProgress, Email?.fetchingCmsSettings, templatesActions, emailActions, params?.id, location?.query, checkBeeEditorEnabled, isFullMode]);
+  }, [Email?.templateDetails, Email?.BEETemplate, Email?.getTemplateDetailsInProgress, Email?.fetchingCmsSettings, templatesActions, emailActions, params?.id, location?.query, checkBeeEditorEnabled, isFullMode, templateData]);
 
   const onChange = useCallback((e) => {
     const { target: { value } } = e;

package/v2Containers/Whatsapp/index.js

@@ -744,7 +744,11 @@ export const Whatsapp = (props) => {
       carouselIndex,
     } = tagSelectData;
     if (mapData && updatedData) {
-      const numId = Number(areaId?.slice(areaId?.indexOf("_") + 1));
+      const underscoreIdx = areaId ? areaId.indexOf("_") : -1;
+      if (underscoreIdx < 0) {
+        return;
+      }
+      const numId = Number(areaId.slice(underscoreIdx + 1));
       if (!isNaN(numId)) {
         const arr = [...updatedData];
         //when trying to insert tag in empty textarea,{#var#} is replaced with "" and then tag is added