From 09fad9764a09fecf9313598973e316aff00d2c9d Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Thu, 31 Oct 2024 11:56:27 +0800 Subject: [PATCH 1/3] Disable posthog --- packages/api/src/utils/analytics.ts | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/packages/api/src/utils/analytics.ts b/packages/api/src/utils/analytics.ts index fa87a31662..081b2ba2c5 100644 --- 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('client') || 'other' - - this.client.capture({ - distinctId, - event, - properties: { - ...properties, - client, - env: env.server.apiEnv, - }, - }) + // // get client from request context + // const client = httpContext.get('client') || 'other' + // this.client.capture({ + // distinctId, + // event, + // properties: { + // ...properties, + // client, + // env: env.server.apiEnv, + // }, + // }) } async shutdownAsync() { From d24059dc0f0f6132f594a1588cf9b38beda14e36 Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Thu, 31 Oct 2024 12:02:17 +0800 Subject: [PATCH 2/3] Disable web posthog --- packages/web/lib/analytics.ts | 24 ++----------------- packages/web/lib/appConfig.ts | 2 -- packages/web/next.config.js | 4 ---- packages/web/package.json | 1 - .../web/pages/[username]/[slug]/index.tsx | 12 ---------- packages/web/pages/_app.tsx | 11 --------- 6 files changed, 2 insertions(+), 52 deletions(-) diff --git a/packages/web/lib/analytics.ts b/packages/web/lib/analytics.ts index db915493dc..1a31268306 100644 --- 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,7 @@ 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) - } -} +export const deinitAnalytics = (): void => {} diff --git a/packages/web/lib/appConfig.ts b/packages/web/lib/appConfig.ts index c9d31a5051..17b3b66df4 100644 --- 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/next.config.js b/packages/web/next.config.js index 4b2d229174..a3b808a482 100644 --- 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 index 32ef416f93..638d10f8f7 100644 --- 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 index ff4f19497a..86f273be95 100644 --- 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 index 1888e18bb1..f3efad2d75 100644 --- 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 ( From e53a3ad9a3a4d1ed54414d3a1249a6d10edda645 Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Thu, 31 Oct 2024 12:09:55 +0800 Subject: [PATCH 3/3] Remove deinit analytics --- packages/web/lib/analytics.ts | 2 -- packages/web/lib/logout.ts | 2 -- 2 files changed, 4 deletions(-) diff --git a/packages/web/lib/analytics.ts b/packages/web/lib/analytics.ts index 1a31268306..a26cb4185a 100644 --- a/packages/web/lib/analytics.ts +++ b/packages/web/lib/analytics.ts @@ -36,5 +36,3 @@ export const setupAnalytics = (user?: UserBasicData): void => { window.Intercom('update', userInfo(user)) } } - -export const deinitAnalytics = (): void => {} diff --git a/packages/web/lib/logout.ts b/packages/web/lib/logout.ts index 058cb6acae..5694a817f1 100644 --- 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')