From 5e3aa163a12e9a3b7ee73e0246a7f06cee9fd203 Mon Sep 17 00:00:00 2001
From: MaheshtheDev <mahesh.svmr@gmail.com>
Date: Fri, 12 Jul 2024 18:27:18 -0700
Subject: [PATCH 1/2] chore: Refactor spaces fetch call activity

---
 apps/extension/content/ContentApp.tsx | 55 +++++++++++++++------------
 1 file changed, 30 insertions(+), 25 deletions(-)

diff --git a/apps/extension/content/ContentApp.tsx b/apps/extension/content/ContentApp.tsx
index d82857e2..217f72ff 100644
--- a/apps/extension/content/ContentApp.tsx
+++ b/apps/extension/content/ContentApp.tsx
@@ -137,35 +137,40 @@ export default function ContentApp({
     shadowRoot.appendChild(portalDiv);
     setPortalContainer(portalDiv);
 
-    const getSpaces = async () => {
-      const response = await fetch(`${BACKEND_URL}/api/spaces`, {
-        headers: {
-          Authorization: `Bearer ${token}`,
-        },
-      });
-
-      if (response.status === 401) {
-        showLoginToast();
-        return;
-      }
-
-      try {
-        const data = await response.json();
-        setSpacesOptions(data.data);
-      } catch (e) {
-        console.error(
-          `Error in supermemory.ai extension: ${e}. Please contact the developer https://x.com/dhravyashah`,
-        );
-      }
-    };
-
-    getSpaces();
-
     return () => {
       document.removeEventListener("mousemove", () => {});
     };
   }, []);
 
+  const getSpaces = async () => {
+    const response = await fetch(`${BACKEND_URL}/api/spaces`, {
+      headers: {
+        Authorization: `Bearer ${token}`,
+      },
+    });
+
+    if (response.status === 401) {
+      showLoginToast();
+      return;
+    }
+
+    try {
+      const data = await response.json();
+      setSpacesOptions(data.data);
+    } catch (e) {
+      console.error(
+        `Error in supermemory.ai extension: ${e}. Please contact the developer https://x.com/dhravyashah`,
+      );
+    }
+  };
+
+  async function handlePopoverChange() {
+    setIsPopoverOpen(!isPopoverOpen);
+    if (isPopoverOpen) {
+      await getSpaces();
+    }
+  }
+
   async function sendUrlToAPI(spaces: string[]) {
     setLoading(true);
 
@@ -243,7 +248,7 @@ export default function ContentApp({
     <div className="flex justify-end items-end min-h-screen h-full w-full">
       <Toaster />
 
-      <Popover open={isPopoverOpen} onOpenChange={setIsPopoverOpen}>
+      <Popover open={isPopoverOpen} onOpenChange={handlePopoverChange}>
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger asChild>

From e0cce43511bde23db0cbf90f4dbf9343960ec835 Mon Sep 17 00:00:00 2001
From: MaheshtheDev <mahesh.svmr@gmail.com>
Date: Thu, 18 Jul 2024 22:20:03 -0700
Subject: [PATCH 2/2] chore: Refactor handlePopoverChange function

---
 apps/extension/content/ContentApp.tsx | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/apps/extension/content/ContentApp.tsx b/apps/extension/content/ContentApp.tsx
index 217f72ff..de760407 100644
--- a/apps/extension/content/ContentApp.tsx
+++ b/apps/extension/content/ContentApp.tsx
@@ -164,13 +164,6 @@ export default function ContentApp({
     }
   };
 
-  async function handlePopoverChange() {
-    setIsPopoverOpen(!isPopoverOpen);
-    if (isPopoverOpen) {
-      await getSpaces();
-    }
-  }
-
   async function sendUrlToAPI(spaces: string[]) {
     setLoading(true);
 
@@ -248,10 +241,13 @@ export default function ContentApp({
     <div className="flex justify-end items-end min-h-screen h-full w-full">
       <Toaster />
 
-      <Popover open={isPopoverOpen} onOpenChange={handlePopoverChange}>
+      <Popover
+        open={isPopoverOpen}
+        onOpenChange={() => setIsPopoverOpen(!isPopoverOpen)}
+      >
         <TooltipProvider>
           <Tooltip>
-            <TooltipTrigger asChild>
+            <TooltipTrigger onClick={async () => await getSpaces()} asChild>
               <PopoverTrigger
                 className={`${hover || isPopoverOpen ? "opacity-100" : "opacity-75 pointer-events-none translate-x-3/4"} focus-within:translate-x-0 focus-visible:translate-x-0 size-12 hover:bg-black p-2 rounded-l-2xl transition bg-secondary border-2 border-border opacity-0 absolute flex bottom-20 items-center text-lg`}
               >