fix: microsoft clarity load and handle promise for identify

rudderlabs · Dec 9, 2024 · 00649a5 · 00649a5
1 parent f946102
commit 00649a5
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/packages/analytics-js-integrations/src/integrations/MicrosoftClarity/browser.js b/packages/analytics-js-integrations/src/integrations/MicrosoftClarity/browser.js
@@ -33,7 +33,7 @@ class MicrosoftClarity {
   }
 
   isLoaded() {
-    return !!window.clarity;
+    return !!window.clarity && window.clarity('identify') instanceof Promise;
   }
 
   isReady() {
@@ -55,14 +55,17 @@ class MicrosoftClarity {
     if (context?.traits?.customPageId) {
       customPageId = context.traits.customPageId;
     }
-    window.clarity('identify', userId, sessionId, customPageId);
-    if (context?.traits) {
-      const { traits } = context;
-      const keys = Object.keys(traits);
-      keys.forEach(key => {
-        window.clarity('set', key, traits[key]);
-      });
-    }
+    window.clarity('identify', userId, sessionId, customPageId).then(() => {
+      if (context?.traits) {
+        const { traits } = context;
+        const keys = Object.keys(traits);
+        keys.forEach(key => {
+          window.clarity('set', key, traits[key]);
+        });
+      }
+    }).catch(error => {
+      logger.error('[MicrosoftClarity] Error in identify call', error);
+    });
   }
 }