fix(frontend): remove intercom on thread page of mobile (#3108)

# Description Please include a summary of the changes and the related issue. Please also include relevant motivation and context. ## Checklist before requesting a review Please delete options that are not relevant. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate):
QuivrHQ · Aug 28, 2024 · a989c05 · a989c05
1 parent b7a081e
commit a989c05
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/frontend/app/layout.tsx b/frontend/app/layout.tsx
@@ -31,10 +31,7 @@ const RootLayout = async ({
 
   return (
     <html lang="en">
-      <body
-        className={styles.body}
-        // className={`bg-white text-black h-screen flex flex-col dark:bg-black dark:text-white w-full ${inter.className}`}
-      >
+      <body className={styles.body}>
         <ToastProvider>
           <SupabaseProvider session={session}>
             <App>{children}</App>

diff --git a/frontend/lib/helpers/intercom/IntercomProvider.ts b/frontend/lib/helpers/intercom/IntercomProvider.ts
@@ -1,11 +1,13 @@
 import { usePathname } from "next/navigation";
 import { useEffect } from "react";
 
+import { useDevice } from "@/lib/hooks/useDevice";
 import { useUserData } from "@/lib/hooks/useUserData";
 
 import {
   boot as bootIntercom,
   load as loadIntercom,
+  shutdown as shutdownIntercom,
   update as updateIntercom,
 } from "./intercom";
 
@@ -16,13 +18,18 @@ export const IntercomProvider = ({
 }): React.ReactNode => {
   const pathname = usePathname();
   const { userData } = useUserData();
+  const { isMobile } = useDevice();
 
   if (
     typeof window !== "undefined" &&
     process.env.NEXT_PUBLIC_INTERCOM_APP_ID
   ) {
-    loadIntercom();
-    bootIntercom(userData?.email ?? "");
+    if (isMobile && pathname?.includes("/chat")) {
+      shutdownIntercom();
+    } else {
+      loadIntercom();
+      bootIntercom(userData?.email ?? "");
+    }
   }
 
   useEffect(() => {

diff --git a/frontend/lib/helpers/intercom/intercom.ts b/frontend/lib/helpers/intercom/intercom.ts
@@ -15,3 +15,7 @@ export const boot = (email: string): void => {
 export const update = (): void => {
   window && (window as any).Intercom && (window as any).Intercom("update");
 };
+
+export const shutdown = (): void => {
+  window && (window as any).Intercom && (window as any).Intercom("shutdown");
+};