fix: send chat id with analytics events (#154)

oaknational · Sep 25, 2024 · 7d201d1 · 7d201d1
1 parent bd2d2eb
commit 7d201d1
Show file tree

Hide file tree

Showing 7 changed files with 55 additions and 19 deletions.
diff --git a/apps/nextjs/src/app/aila/[id]/download/DownloadView.tsx b/apps/nextjs/src/app/aila/[id]/download/DownloadView.tsx
@@ -17,7 +17,7 @@ type DownloadViewProps = Readonly<{
   chat: AilaPersistedChat;
 }>;
 export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
-  const { lessonPlan } = chat;
+  const { lessonPlan, id } = chat;
   const {
     lessonSlidesExport,
     worksheetExport,
@@ -82,6 +82,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
               >
                 <Flex direction="column" className="gap-14">
                   <DownloadButton
+                    chatId={id}
                     onClick={() => lessonPlanExport.start()}
                     title="Lesson plan"
                     subTitle="Overview of the complete lesson"
@@ -93,6 +94,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
                     lesson={lessonPlan}
                   />
                   <DownloadButton
+                    chatId={id}
                     onClick={() => starterQuizExport.start()}
                     title="Starter quiz"
                     subTitle="Questions and answers to assess prior knowledge"
@@ -103,6 +105,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
                     lesson={lessonPlan}
                   />
                   <DownloadButton
+                    chatId={id}
                     onClick={() => lessonSlidesExport.start()}
                     data-testid="chat-download-slides-btn"
                     title="Slide deck"
@@ -114,6 +117,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
                     lesson={lessonPlan}
                   />
                   <DownloadButton
+                    chatId={id}
                     onClick={() => worksheetExport.start()}
                     title="Worksheet"
                     subTitle="Practice tasks"
@@ -124,6 +128,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
                     exportsType="worksheet"
                   />
                   <DownloadButton
+                    chatId={id}
                     onClick={() => exitQuizExport.start()}
                     title="Exit quiz"
                     subTitle="Questions and answers to assess understanding"
@@ -136,6 +141,7 @@ export function DownloadView({ chat }: Readonly<DownloadViewProps>) {
                   {lessonPlan.additionalMaterials &&
                     lessonPlan.additionalMaterials !== "None" && (
                       <DownloadButton
+                        chatId={id}
                         onClick={() => additionalMaterialsExport.start()}
                         title="Additional materials"
                         subTitle="Document containing any additional materials"

diff --git a/apps/nextjs/src/components/DialogControl/ContentOptions/ShareChatDialog.tsx b/apps/nextjs/src/components/DialogControl/ContentOptions/ShareChatDialog.tsx
@@ -36,6 +36,7 @@ const ShareChat = ({
   const attemptToShareChat = async () => {
     track.lessonPlanShared({
       ...getLessonTrackingProps({ lesson }),
+      chatId,
       componentType: "go_to_share_page_button",
     });
     if (isShared) {

diff --git a/apps/nextjs/src/components/DownloadButton.tsx b/apps/nextjs/src/components/DownloadButton.tsx
@@ -18,6 +18,7 @@ import QuizIcon from "./SVGParts/QuizIcon";
 import SlidesIcon from "./SVGParts/SlidesIcon";
 
 export const DownloadButton = ({
+  chatId,
   onClick,
   lesson,
   title,
@@ -28,6 +29,7 @@ export const DownloadButton = ({
   exportsType,
   "data-testid": dataTestId,
 }: {
+  chatId: string;
   onClick: () => void;
   lesson: LooseLessonPlan;
   title: string;
@@ -51,6 +53,7 @@ export const DownloadButton = ({
 
   function trackDownload(resourceFileType: ResourceFileTypeValueType) {
     track.lessonPlanResourcesDownloaded({
+      chatId,
       ...getLessonTrackingProps({ lesson }),
       resourceType: [analyticsResourceType],
       resourceFileType,

diff --git a/apps/nextjs/src/lib/analytics/lessonPlanTrackingContext.tsx b/apps/nextjs/src/lib/analytics/lessonPlanTrackingContext.tsx
@@ -35,7 +35,7 @@ const lessonPlanTrackingContext =
 
 const LessonPlanTrackingProvider: FC<{
   children?: React.ReactNode;
-  chatId?: string;
+  chatId: string;
 }> = ({ children, chatId }) => {
   const { track } = useAnalytics();
   const [action, setAction] = useState<UserAction | null>(null);
@@ -51,6 +51,7 @@ const LessonPlanTrackingProvider: FC<{
       }
       const isFirstMessage = messages.length === 2;
       trackLessonPlanRefined({
+        chatId,
         prevLesson,
         nextLesson,
         ailaMessageContent,

diff --git a/apps/nextjs/src/lib/analytics/trackLessonPlanRefined.ts b/apps/nextjs/src/lib/analytics/trackLessonPlanRefined.ts
@@ -45,6 +45,7 @@ function actionToComponentType(
  * changes, we still need to track the lesson plan as it progresses.
  */
 export function trackLessonPlanRefined({
+  chatId,
   prevLesson,
   nextLesson,
   userMessageContent,
@@ -53,6 +54,7 @@ export function trackLessonPlanRefined({
   track,
   action,
 }: {
+  chatId: string;
   prevLesson: LooseLessonPlan;
   nextLesson: LooseLessonPlan;
   userMessageContent: string;
@@ -75,6 +77,7 @@ export function trackLessonPlanRefined({
   if (isFirstMessage) {
     track.lessonPlanInitiated({
       ...getLessonTrackingProps({ lesson: nextLesson }),
+      chatId,
       moderatedContentType: getModerationTypes(moderation),
       text: userMessageContent,
       componentType,
@@ -89,6 +92,7 @@ export function trackLessonPlanRefined({
     );
     track.lessonPlanRefined({
       ...getLessonTrackingProps({ lesson: nextLesson }),
+      chatId,
       moderatedContentType: getModerationTypes(moderation),
       text: userMessageContent,
       componentType: isSelectingOakLesson ? "select_oak_lesson" : componentType,
@@ -104,6 +108,7 @@ export function trackLessonPlanRefined({
   if (!isLessonComplete(prevLesson) && isLessonComplete(nextLesson)) {
     track.lessonPlanCompleted({
       ...getLessonTrackingProps({ lesson: nextLesson }),
+      chatId,
       moderatedContentType: getModerationTypes(moderation),
     });
   }
@@ -113,6 +118,7 @@ export function trackLessonPlanRefined({
   const isTerminated = accountLocked || (moderation && isToxic(moderation));
   if (isTerminated) {
     track.lessonPlanTerminated({
+      chatId,
       isUserBlocked: accountLocked,
       isToxicContent: moderation ? isToxic(moderation) : false,
       isThreatDetected: false, // @fixme currently we can't access threat detection here