Fixing the Trigger of Heap Event : Console Load Onboarding Wizard

PR-URL: hasura/graphql-engine-mono#9193 GitOrigin-RevId: 922940c5c3fbfb08efe0dd838d1e0e2c63459e53
hasura · May 18, 2023 · 369bfb7 · 369bfb7
1 parent 07c08e7
commit 369bfb7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/.../features/CloudOnboarding/OnboardingWizard/components/ConnectDBScreen/ConnectDBScreen.tsx b/.../features/CloudOnboarding/OnboardingWizard/components/ConnectDBScreen/ConnectDBScreen.tsx
@@ -1,8 +1,9 @@
-import React from 'react';
+import React, { useEffect } from 'react';
 import { Dispatch } from '../../../../../types';
 
 import { OnboardingAnimation } from './components/OnboardingAnimation';
 import { NeonOnboarding } from './components/NeonOnboarding';
+import { trackCustomEvent } from '../../../../Analytics';
 
 type ConnectDBScreenProps = {
   proceed: VoidFunction;
@@ -12,6 +13,13 @@ type ConnectDBScreenProps = {
 };
 
 export function ConnectDBScreen(props: ConnectDBScreenProps) {
+  useEffect(() => {
+    trackCustomEvent({
+      location: 'Console',
+      action: 'Load',
+      object: 'Neon Onboarding Wizard',
+    });
+  }, []);
   const { proceed, dismissOnboarding, dispatch, setStepperIndex } = props;
 
   return (

diff --git a/...nsole/legacy-ce/src/lib/features/CloudOnboarding/OnboardingWizard/hooks/useWizardState.ts b/...nsole/legacy-ce/src/lib/features/CloudOnboarding/OnboardingWizard/hooks/useWizardState.ts
@@ -1,6 +1,5 @@
 import { useEffect, useState } from 'react';
 import { useSurveysData } from '../../../Surveys';
-import { trackCustomEvent } from '../../../Analytics';
 import { useOnboardingData } from './useOnboardingData';
 import { getWizardState } from '../utils';
 import { AllowedSurveyNames } from '../../../Surveys/types';
@@ -14,14 +13,6 @@ export type WizardState =
   | 'hidden';
 
 export function useWizardState() {
-  useEffect(() => {
-    trackCustomEvent({
-      location: 'Console',
-      action: 'Load',
-      object: 'Onboarding Wizard',
-    });
-  }, []);
-
   const {
     show: showSurvey,
     data: familiaritySurveyData,