Smaller fixes

Signed-off-by: Heiko W. Rupp <[email protected]>

Author: pilhuhn

src/app/playground/chat/page.tsx

@@ -36,7 +36,6 @@ const ChatPage: React.FC = () => {
           variant="link"
           isInline
           onClick={(e) => {
-            window.analytics.trackSingleItem('Chat Cleared', {});
             e.preventDefault();
             router.push('/playground/endpoints');
           }}

src/components/AppLayout.tsx

@@ -48,6 +48,7 @@ type Route = {
 const AppLayout: React.FunctionComponent<IAppLayout> = ({ children, className }) => {
   const { data: session, status } = useSession();
   const [isExperimentalEnabled, setExperimental] = useState(false);
+  const [analyticsInitialised, setAnalyticsInitialised] = useState(false);
 
   const router = useRouter();
   const pathname = usePathname();
@@ -63,9 +64,9 @@ const AppLayout: React.FunctionComponent<IAppLayout> = ({ children, className })
   }, []);
 
   React.useEffect(() => {
-    console.log("Get analytics effect " + window.analytics);
     if (!window.analytics) {
       initAnalytics();
+      setAnalyticsInitialised(true);
     }
   }, []);
 
@@ -74,15 +75,16 @@ const AppLayout: React.FunctionComponent<IAppLayout> = ({ children, className })
       window.analytics.trackPageView(pathname);
     }
 
-  }, [pathname]);
+  }, [pathname, analyticsInitialised]);
 
   React.useEffect(() => {
-    console.log(("Identify effect " + session?.user) )
     if (window.analytics) {
       // TODO we may potentially want to hash this. Also different code per target install?
-      window.analytics.identify(session?.user?.name ? session.user.name : '-unknown-user-name ');
+      // TODO pass other parameters as properties
+      window.analytics.identify(session?.user?.name ? session.user.name : '-unknown-user-name ', {}
+      );
     }
-  },[session?.user?.name,session?.user]);
+  },[analyticsInitialised, session?.user?.name,session?.user]);
 
   React.useEffect(() => {
     if (status === 'loading') return; // Do nothing while loading

src/components/Chat/ChatBotComponent.tsx

@@ -214,6 +214,7 @@ const ChatBotComponent: React.FunctionComponent<ChatbotComponentProps> = ({
 
   const handleCleanup = () => {
     setMessages([]);
+    window.analytics.trackSingleItem('Chat Cleared', {});
   };
 
   return (

src/components/Contribute/ContributionWizard/ContributionWizard.tsx

@@ -189,7 +189,10 @@ export const ContributionWizard: React.FunctionComponent<Props> = ({
           style={{ maxHeight: '100%' }}
           startIndex={1}
           onClose={handleCancel}
-          onStepChange={(_ev, currentStep) => setActiveStepIndex(stepIds.indexOf(String(currentStep.id)))}
+          onStepChange={(_ev, currentStep) => {
+            setActiveStepIndex(stepIds.indexOf(String(currentStep.id)));
+            window.analytics.trackSingleItem('Wizard Step Changed', { isSkillContribution, step: currentStep.id });}
+          }
           footer={
             <ContributionWizardFooter
               onCancel={handleCancel}

src/components/Contribute/Skill/SkillWizard/SkillWizard.tsx

@@ -176,7 +176,7 @@ export const SkillWizard: React.FunctionComponent<Props> = ({ skillEditFormData,
   };
 
   const handleSubmit = async (githubUsername: string): Promise<boolean> => {
-    window.analytics.trackSingleItem("Knowledge Contribution Submitted", {}); // TODO better place, add data
+    window.analytics.trackSingleItem("Skill Contribution Submitted", {}); // TODO better place, add data
     if (skillEditFormData) {
       const result = isGithubMode
         ? await updateGithubSkillData(session, skillFormData, skillEditFormData, updateActionGroupAlertContent)