📈 send answer in question event (#861)

incubateur-ademe · Feb 19, 2025 · fb3be26 · fb3be26
1 parent 6378b2d
commit fb3be26
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 33 deletions.
diff --git a/src/components/form/Question.tsx b/src/components/form/Question.tsx
@@ -12,10 +12,7 @@ import {
   DEFAULT_FOCUS_ELEMENT_ID,
   QUESTION_DESCRIPTION_BUTTON_ID,
 } from '@/constants/accessibility'
-import {
-  questionChooseAnswer,
-  questionTypeAnswer,
-} from '@/constants/tracking/question'
+import { questionChooseAnswer } from '@/constants/tracking/question'
 import Button from '@/design-system/inputs/Button'
 import { useRule } from '@/publicodes-state'
 import { trackEvent } from '@/utils/analytics/trackEvent'
@@ -109,7 +106,6 @@ export default function Question({
                     setTempValue(value)
                   }
                   setValue(value, { questionDottedName: question })
-                  trackEvent(questionTypeAnswer({ question, answer: value }))
                 }}
                 isMissing={isMissing}
                 min={0}

diff --git a/src/components/form/question/mosaic/MosaicQuestion.tsx b/src/components/form/question/mosaic/MosaicQuestion.tsx
@@ -1,9 +1,6 @@
 'use client'
 
-import {
-  questionChooseAnswer,
-  questionTypeAnswer,
-} from '@/constants/tracking/question'
+import { questionChooseAnswer } from '@/constants/tracking/question'
 import { useRule } from '@/publicodes-state'
 import { trackEvent } from '@/utils/analytics/trackEvent'
 import type { DottedName } from '@incubateur-ademe/nosgestesclimat'
@@ -40,14 +37,6 @@ export default function MosaicQuestion({
               questionDottedName: parentMosaic,
               questionsOfMosaicFromSibling,
             })
-
-            trackEvent(
-              questionTypeAnswer({
-                question: parentMosaic,
-                answer: parent,
-                mosaicValue: value,
-              })
-            )
           }}
           parentMosaic={parentMosaic}
           index={index}
@@ -68,9 +57,8 @@ export default function MosaicQuestion({
 
             trackEvent(
               questionChooseAnswer({
-                question: parentMosaic,
-                answer: parent,
-                mosaicValue: value,
+                question: parent,
+                answer: value,
               })
             )
           }}

diff --git a/src/constants/tracking/layout.ts b/src/constants/tracking/layout.ts
@@ -99,4 +99,9 @@ export const footerClickLanguage = (locale: string) => [
 ]
 
 // Figma comment #18
-export const breadcrumbClickLink = ['trackEvent', 'Breadcrumb', 'Click link']
+export const breadcrumbClickLink = (href: string) => [
+  'trackEvent',
+  'Breadcrumb',
+  'Click link',
+  href,
+]
diff --git a/src/constants/tracking/question.ts b/src/constants/tracking/question.ts
@@ -6,7 +6,6 @@ import type { DottedName, NodeValue } from '@incubateur-ademe/nosgestesclimat'
 type Props = {
   question: DottedName
   answer?: DottedName | NodeValue | string
-  mosaicValue?: NodeValue | string
   timeSpentOnQuestion?: number
 }
 // Figma comment #45
@@ -55,22 +54,23 @@ export const questionClickPrevious = ({ question }: Props) => [
 ]
 
 // Figma comment #46
-export const questionChooseAnswer = ({ question, mosaicValue }: Props) => [
+export const questionChooseAnswer = ({ question, answer }: Props) => [
   'trackEvent',
   'Simulateur',
   'Choose Answer',
   question,
-  String(mosaicValue),
+  String(answer),
 ]
 
 // Figma comment #51
-export const questionTypeAnswer = ({ question, mosaicValue }: Props) => [
-  'trackEvent',
-  'Simulateur',
-  'Type Answer',
-  question,
-  String(mosaicValue),
-]
+// An event is sent on every change of the input, it's too much !
+// export const questionTypeAnswer = ({ question, answer }: Props) => [
+//   'trackEvent',
+//   'Simulateur',
+//   'Type Answer',
+//   question,
+//   String(answer),
+// ]
 
 // Figma comment #50
 export const questionClickSuggestion = ({ question, answer }: Props) => [

diff --git a/src/design-system/layout/Breadcrumbs.tsx b/src/design-system/layout/Breadcrumbs.tsx
@@ -30,7 +30,7 @@ export default function Breadcrumbs({
                 if (isDisabled) {
                   e.preventDefault()
                 }
-                trackEvent(breadcrumbClickLink)
+                trackEvent(breadcrumbClickLink(href))
               }}
               aria-current={isActive}
               className={twMerge(

diff --git a/src/utils/analytics/trackEvent.ts b/src/utils/analytics/trackEvent.ts
@@ -10,7 +10,6 @@ declare global {
 
 export const trackEvent = (args: (string | null)[]) => {
   if (shouldUseDevTracker || !window?._paq) {
-    console.log(args)
     console.debug(args.join(' => '))
     return
   }