Merge pull request #4466 from omnivore-app/jacksonh/disable-posthog

Disable posthog
omnivore-app · Oct 31, 2024 · ccaccab · ccaccab
2 parents c5343b1 + e53a3ad
commit ccaccab
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 67 deletions.
diff --git a/packages/api/src/utils/analytics.ts b/packages/api/src/utils/analytics.ts
@@ -21,18 +21,17 @@ class PostHogClient implements AnalyticClient {
   }
 
   capture({ distinctId, event, properties }: AnalyticEvent) {
-    // get client from request context
-    const client = httpContext.get<string>('client') || 'other'
-
-    this.client.capture({
-      distinctId,
-      event,
-      properties: {
-        ...properties,
-        client,
-        env: env.server.apiEnv,
-      },
-    })
+    // // get client from request context
+    // const client = httpContext.get<string>('client') || 'other'
+    // this.client.capture({
+    //   distinctId,
+    //   event,
+    //   properties: {
+    //     ...properties,
+    //     client,
+    //     env: env.server.apiEnv,
+    //   },
+    // })
   }
 
   async shutdownAsync() {

diff --git a/packages/web/lib/analytics.ts b/packages/web/lib/analytics.ts
@@ -1,6 +1,5 @@
 import { UserBasicData } from './networking/queries/useGetViewerQuery'
-import { intercomAppID, posthogApiKey, webBaseURL } from './appConfig'
-import posthog from 'posthog-js'
+import { intercomAppID } from './appConfig'
 
 const userInfo = (
   user: UserBasicData
@@ -19,17 +18,6 @@ const initAnalytics = (user?: UserBasicData): void => {
     vertical_padding: 120,
     custom_launcher_selector: '.custom-intercom-launcher',
   }
-  if (posthogApiKey) {
-    posthog.init(posthogApiKey, {
-      api_host: `${webBaseURL}/collect`,
-      autocapture: false,
-      person_profiles: 'identified_only',
-      disable_session_recording: false,
-      advanced_disable_decide: true,
-      advanced_disable_feature_flags: true,
-      advanced_disable_toolbar_metrics: true,
-    })
-  }
   if (user) {
     window.Intercom('boot', userInfo(user))
   }
@@ -46,15 +34,5 @@ export const setupAnalytics = (user?: UserBasicData): void => {
 
   if (user) {
     window.Intercom('update', userInfo(user))
-    posthog.identify(user.id, {
-      name: user.name,
-      username: user.profile.username,
-    })
-  }
-}
-
-export const deinitAnalytics = (): void => {
-  if (posthog && posthogApiKey) {
-    posthog.reset(true)
   }
 }
diff --git a/packages/web/lib/appConfig.ts b/packages/web/lib/appConfig.ts
@@ -70,8 +70,6 @@ export const appleAuthRedirectURI =
 
 export const intercomAppID = process.env.NEXT_PUBLIC_INTERCOM_APP_ID
 
-export const posthogApiKey = process.env.NEXT_PUBLIC_POSTHOG_API_KEY
-
 export const googleID =
   appEnv == 'prod'
     ? process.env.NEXT_PUBLIC_GOOGLE_ID

diff --git a/packages/web/lib/logout.ts b/packages/web/lib/logout.ts
@@ -1,5 +1,4 @@
 import { useQueryClient } from '@tanstack/react-query'
-import { deinitAnalytics } from './analytics'
 import { logoutMutation } from './networking/mutations/logoutMutation'
 import { useCallback } from 'react'
 import { useRouter } from 'next/router'
@@ -14,7 +13,6 @@ export const useLogout = () => {
       if (!result) {
         throw new Error('Logout failed')
       }
-      deinitAnalytics()
       queryClient.clear()
       console.log('cleared the query client')
       router.push('/login')

diff --git a/packages/web/next.config.js b/packages/web/next.config.js
@@ -45,10 +45,6 @@ const moduleExports = {
         }
       )
     }
-    rewrites.push({
-      source: '/collect/:match*',
-      destination: 'https://app.posthog.com/:match*',
-    })
     rewrites.push({
       source: '/home',
       destination: '/l/home',

diff --git a/packages/web/package.json b/packages/web/package.json
@@ -52,7 +52,6 @@
     "next": "^13.5.6",
     "node-html-markdown": "^1.3.0",
     "papaparse": "^5.4.1",
-    "posthog-js": "^1.158.3",
     "pspdfkit": "^2023.4.6",
     "re-resizable": "^6.9.11",
     "react": "^18.2.0",

diff --git a/packages/web/pages/[username]/[slug]/index.tsx b/packages/web/pages/[username]/[slug]/index.tsx
@@ -21,7 +21,6 @@ import { VerticalArticleActionsMenu } from '../../../components/templates/articl
 import { PdfHeaderSpacer } from '../../../components/templates/article/PdfHeaderSpacer'
 import { EpubContainerProps } from '../../../components/templates/article/EpubContainer'
 import { useSetPageLabels } from '../../../lib/hooks/useSetPageLabels'
-import { posthog } from 'posthog-js'
 import { PDFDisplaySettingsModal } from '../../../components/templates/article/PDFDisplaySettingsModal'
 import {
   ArticleReadingProgressMutationInput,
@@ -270,17 +269,6 @@ export default function Reader(): JSX.Element {
     }
   }, [actionHandler, goNextOrHome, goPreviousOrHome])
 
-  useEffect(() => {
-    if (libraryItem && viewerData) {
-      posthog.capture('link_read', {
-        link: libraryItem.id,
-        slug: libraryItem.slug,
-        reader: libraryItem.contentReader,
-        url: libraryItem.originalArticleUrl,
-      })
-    }
-  }, [libraryItem, viewerData])
-
   useRegisterActions(
     [
       {

diff --git a/packages/web/pages/_app.tsx b/packages/web/pages/_app.tsx
@@ -22,7 +22,6 @@ import {
 } from '../components/elements/KBar'
 import { updateTheme } from '../lib/themeUpdater'
 import { ThemeId } from '../components/tokens/stitches.config'
-import { posthog } from 'posthog-js'
 import { GoogleReCaptchaProvider } from '@google-recaptcha/react'
 import AsyncStorage from '@react-native-async-storage/async-storage'
 import { QueryClient } from '@tanstack/react-query'
@@ -96,16 +95,6 @@ const ConditionalCaptchaProvider = (props: {
 export function OmnivoreApp({ Component, pageProps }: AppProps): JSX.Element {
   const router = useRouter()
 
-  useEffect(() => {
-    const handleRouteChange = (url: string) => {
-      posthog.capture('$pageview')
-    }
-    router.events.on('routeChangeComplete', handleRouteChange)
-    return () => {
-      router.events.off('routeChangeComplete', handleRouteChange)
-    }
-  }, [router.events])
-
   return (
     <ConditionalCaptchaProvider>
       <Toaster />