From 2fb25d1379fcb42ed3fc6bf9d0ea80278c8c1e02 Mon Sep 17 00:00:00 2001
From: Mike <mike.tasset@gmail.com>
Date: Mon, 16 Sep 2024 14:34:30 +0200
Subject: [PATCH] Fix: Change add_group to people.set for workspaces (#2999)

---
 .../components/dashboard/Sidebar.vue          | 19 ++++++++++++++++---
 .../frontend-2/components/invite/Banner.vue   |  1 -
 .../invite/DiscoverableWorkspaceBanner.vue    |  1 -
 .../lib/workspaces/composables/management.ts  |  1 -
 4 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/packages/frontend-2/components/dashboard/Sidebar.vue b/packages/frontend-2/components/dashboard/Sidebar.vue
index 88dcac0be..360e464e6 100644
--- a/packages/frontend-2/components/dashboard/Sidebar.vue
+++ b/packages/frontend-2/components/dashboard/Sidebar.vue
@@ -203,9 +203,13 @@ const mixpanel = useMixpanel()
 const isOpenMobile = ref(false)
 const showWorkspaceCreateDialog = ref(false)
 
-const { result: workspaceResult } = useQuery(settingsSidebarQuery, null, {
-  enabled: isWorkspacesEnabled.value
-})
+const { result: workspaceResult, onResult: onWorkspaceResult } = useQuery(
+  settingsSidebarQuery,
+  null,
+  {
+    enabled: isWorkspacesEnabled.value
+  }
+)
 
 const isActive = (...routes: string[]): boolean => {
   return routes.some((routeTo) => route.path === routeTo)
@@ -233,4 +237,13 @@ const openWorkspaceCreateDialog = () => {
     source: 'sidebar'
   })
 }
+
+onWorkspaceResult((result) => {
+  if (result.data?.activeUser) {
+    const workspaceIds = result.data.activeUser.workspaces.items.map(
+      (workspace) => workspace.id
+    )
+    mixpanel.people.set('workspace_id', workspaceIds)
+  }
+})
 </script>
diff --git a/packages/frontend-2/components/invite/Banner.vue b/packages/frontend-2/components/invite/Banner.vue
index 143662532..0d605747a 100644
--- a/packages/frontend-2/components/invite/Banner.vue
+++ b/packages/frontend-2/components/invite/Banner.vue
@@ -171,7 +171,6 @@ const onAcceptClick = (token?: string) => {
       // eslint-disable-next-line camelcase
       workspace_id: props.invite.workspace.id
     })
-    mixpanel.add_group('workspace_id', props.invite.workspace.id)
   }
 }
 </script>
diff --git a/packages/frontend-2/components/workspace/invite/DiscoverableWorkspaceBanner.vue b/packages/frontend-2/components/workspace/invite/DiscoverableWorkspaceBanner.vue
index 91a3c1b81..ff8963d66 100644
--- a/packages/frontend-2/components/workspace/invite/DiscoverableWorkspaceBanner.vue
+++ b/packages/frontend-2/components/workspace/invite/DiscoverableWorkspaceBanner.vue
@@ -129,7 +129,6 @@ const processJoin = async (accept: boolean) => {
       // eslint-disable-next-line camelcase
       workspace_id: props.workspace.id
     })
-    mixpanel.add_group('workspace_id', props.workspace.id)
 
     router.push(`/workspaces/${props.workspace.id}`)
   } else {
diff --git a/packages/frontend-2/lib/workspaces/composables/management.ts b/packages/frontend-2/lib/workspaces/composables/management.ts
index 3154be36a..45de0e689 100644
--- a/packages/frontend-2/lib/workspaces/composables/management.ts
+++ b/packages/frontend-2/lib/workspaces/composables/management.ts
@@ -196,7 +196,6 @@ export const useProcessWorkspaceInvite = () => {
         // eslint-disable-next-line camelcase
         workspace_id: workspaceId
       })
-      mp.add_group('workspace_id', workspaceId)
     } else {
       const err = getFirstErrorMessage(errors)
       const preventErrorToasts = isFunction(options?.preventErrorToasts)