Merge pull request #8115 from supabase/fix/telemtry-signup

Update PageTelemetry.tsx

studio/components/ui/PageTelemetry.tsx

@@ -23,16 +23,34 @@ const PageTelemetry: FC = ({ children }) => {
     }
   }, [router.events, profile])
 
+  useEffect(() => {
+    /**
+     * Send page telemetry on first page load
+     */
+    handlePageTelemetry(profile)
+  }, [])
+
   const handlePageTelemetry = (profile?: User) => {
     if (IS_PLATFORM) {
-      // exclude github.com from referrer if user is logged in
-      // add other auth providers in here if necessary
-      const referrer =
-        !isUndefined(profile) && document.referrer === 'https://github.com' ? '' : document.referrer
+      /**
+       * Get referrer from browser
+       */
+      let referrer: string | undefined = document.referrer
 
-      // send page view
-      // TODO: document.title is lagging behind routeChangeComplete
-      // that means the page title is the previous one instead of the new page title
+      /**
+       * When user is logging in for first time, callback url will contain param `signup=true`
+       * If true, we will exclude the referrer, which will likely be `github.com` (or any other provider used in future)
+       */
+      if (router.asPath && router.asPath.includes('signup=true')) {
+        referrer = undefined
+      }
+
+      /**
+       * Send page telemetry
+       *
+       * TODO: document.title is lagging behind routeChangeComplete
+       * that means the page title is the previous one instead of the new page title
+       */
       post(`${API_URL}/telemetry/page`, {
         referrer: referrer,
         title: document.title,