fix api call

packages/gitbook/src/components/PageBody/PageBody.tsx

@@ -149,7 +149,6 @@ export function PageBody(props: {
             {shouldTrackPageViews() ? (
                 <TrackPageView
                     sitePointer={pointer}
-                    spaceId={space.id}
                     pageId={page.id}
                     apiHost={api().client.endpoint}
                 />

packages/gitbook/src/components/PageBody/TrackPageView.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import type { RequestSiteTrackPageView, RequestSpaceTrackPageView } from '@gitbook/api';
+import type { RequestSiteTrackPageView } from '@gitbook/api';
 import cookies from 'js-cookie';
 import * as React from 'react';
 
@@ -13,36 +13,17 @@ import { SiteContentPointer } from '@/lib/api';
 export function TrackPageView(props: {
     apiHost: string;
     sitePointer: SiteContentPointer;
-    spaceId: string;
     pageId: string | undefined;
 }) {
-    const { apiHost, sitePointer, spaceId, pageId } = props;
+    const { apiHost, sitePointer, pageId } = props;
 
     React.useEffect(() => {
-        trackPageView({ apiHost, sitePointer, spaceId, pageId });
-    }, [apiHost, spaceId, pageId, sitePointer]);
+        trackPageView({ apiHost, sitePointer, pageId });
+    }, [apiHost, pageId, sitePointer]);
 
     return null;
 }
 
-async function sendSpaceTrackPageViewRequest(args: {
-    apiHost: string;
-    spaceId: string;
-    body: RequestSpaceTrackPageView;
-}) {
-    const { apiHost, spaceId, body } = args;
-    const url = new URL(apiHost);
-    url.pathname = `/v1/spaces/${spaceId}/insights/track_view`;
-
-    await fetch(url, {
-        method: 'POST',
-        headers: {
-            'Content-Type': 'application/json',
-        },
-        body: JSON.stringify(body),
-    });
-}
-
 async function sendSiteTrackPageViewRequest(args: {
     apiHost: string;
     sitePointer: SiteContentPointer;
@@ -71,10 +52,9 @@ let latestPageId: string | undefined | null = null;
 async function trackPageView(args: {
     apiHost: string;
     sitePointer: SiteContentPointer;
-    spaceId: string;
     pageId: string | undefined;
 }) {
-    const { apiHost, sitePointer, pageId, spaceId } = args;
+    const { apiHost, sitePointer, pageId } = args;
     if (pageId === latestPageId) {
         // The hook can be called multiple times, we only want to track once.
         return;
@@ -96,16 +76,17 @@ async function trackPageView(args: {
     };
 
     try {
-        sitePointer
-            ? await sendSiteTrackPageViewRequest({
-                  apiHost,
-                  sitePointer,
-                  body: {
-                      ...sharedTrackedProps,
-                      spaceId,
-                  },
-              })
-            : await sendSpaceTrackPageViewRequest({ apiHost, spaceId, body: sharedTrackedProps });
+        // siteSpaceId is only undefined in mult-id mode (site previews) so we skip the tracking for those
+        if (sitePointer.siteSpaceId) {
+            await sendSiteTrackPageViewRequest({
+                apiHost,
+                sitePointer,
+                body: {
+                    ...sharedTrackedProps,
+                    siteSpaceId: sitePointer.siteSpaceId,
+                },
+            });
+        }
     } catch (error) {
         console.error('Failed to track page view', error);
     }