feat(CRater): Ping endpoint when component is rendered (#2236)

hirokiterashima · web-flow · commit ab4652eccd04 · 2025-10-08T23:37:50.000-07:00
diff --git a/src/assets/wise5/components/dialogGuidance/dialog-guidance-student/dialog-guidance-student.component.html b/src/assets/wise5/components/dialogGuidance/dialog-guidance-student/dialog-guidance-student.component.html
@@ -14,7 +14,6 @@
       />
       @if (studentCanRespond) {
         <chat-input
-          (focusEvent)="startPinging()"
           [submitDisabled]="isWaitingForComputerResponse"
           (submitEvent)="submitStudentResponse($event)"
         />
diff --git a/src/assets/wise5/components/dialogGuidance/dialog-guidance-student/dialog-guidance-student.component.ts b/src/assets/wise5/components/dialogGuidance/dialog-guidance-student/dialog-guidance-student.component.ts
@@ -108,9 +108,10 @@ export class DialogGuidanceStudentComponent extends ComponentStudent {
       this.constraintService
     );
     this.initializeComputerAvatar();
+    this.pingCRaterEndpoint();
   }
 
-  protected startPinging(): void {
+  private pingCRaterEndpoint(): void {
     this.cRaterPingService.startPinging(this.getItemId());
   }
 
diff --git a/src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.html b/src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.html
@@ -34,7 +34,6 @@
       (ngModelChange)="studentDataChanged()"
       [disabled]="isDisabled"
       cdkTextareaAutosize
-      (focus)="onFocus()"
     >
     </textarea>
   </mat-form-field>
diff --git a/src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts b/src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts
@@ -107,10 +107,11 @@ export class OpenResponseStudent extends ComponentStudent {
       });
     }
     this.updateAudioAttachments();
+    this.pingCRaterEndpoint();
     this.broadcastDoneRenderingComponent();
   }
 
-  protected onFocus() {
+  private pingCRaterEndpoint(): void {
     if (this.isCRaterEnabled()) {
       this.cRaterPingService.startPinging(this.getItemId());
     }
diff --git a/src/messages.xlf b/src/messages.xlf
@@ -16542,7 +16542,7 @@ Are you sure you want to proceed?</source>
         </context-group>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.html</context>
-          <context context-type="linenumber">58,60</context>
+          <context context-type="linenumber">57,59</context>
         </context-group>
       </trans-unit>
       <trans-unit id="7589669206989444799" datatype="html">
@@ -16553,11 +16553,11 @@ Are you sure you want to proceed?</source>
         </context-group>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">179</context>
+          <context context-type="linenumber">180</context>
         </context-group>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">195</context>
+          <context context-type="linenumber">196</context>
         </context-group>
       </trans-unit>
       <trans-unit id="4965414112249568013" datatype="html">
@@ -17494,7 +17494,7 @@ Are you ready to receive feedback on this answer?</source>
         </context-group>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">182</context>
+          <context context-type="linenumber">183</context>
         </context-group>
       </trans-unit>
       <trans-unit id="2353574516166744013" datatype="html">
@@ -20491,7 +20491,7 @@ Current Score: <x id="PH_1" equiv-text="currentScore"/></source>
 Are you ready to receive feedback on this answer?</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">186</context>
+          <context context-type="linenumber">187</context>
         </context-group>
       </trans-unit>
       <trans-unit id="7482125205220618294" datatype="html">
@@ -20500,7 +20500,7 @@ Are you ready to receive feedback on this answer?</source>
 Are you ready to receive feedback on this answer?</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">198</context>
+          <context context-type="linenumber">199</context>
         </context-group>
       </trans-unit>
       <trans-unit id="6316287012266891697" datatype="html">
@@ -20509,29 +20509,29 @@ Are you ready to receive feedback on this answer?</source>
 Are you ready to submit this answer?</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">202</context>
+          <context context-type="linenumber">203</context>
         </context-group>
       </trans-unit>
       <trans-unit id="8936495196157450285" datatype="html">
         <source>We are scoring your work...</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">321</context>
+          <context context-type="linenumber">322</context>
         </context-group>
       </trans-unit>
       <trans-unit id="2283191014272031538" datatype="html">
         <source>Please Wait</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">322</context>
+          <context context-type="linenumber">323</context>
         </context-group>
       </trans-unit>
       <trans-unit id="8785209093929133444" datatype="html">
         <source>There was an issue scoring your work. Please try again.
 If this problem continues, let your teacher know and move on to the next activity. Your work will still be saved.</source>
         <context-group purpose="location">
           <context context-type="sourcefile">src/assets/wise5/components/openResponse/open-response-student/open-response-student.component.ts</context>
-          <context context-type="linenumber">344</context>
+          <context context-type="linenumber">345</context>
         </context-group>
       </trans-unit>
       <trans-unit id="1703970408358128709" datatype="html">

Original file line number	Diff line number	Diff line change
`@@ -108,9 +108,10 @@ export class DialogGuidanceStudentComponent extends ComponentStudent {`
`108`	`108`	`this.constraintService`
`109`	`109`	`);`
`110`	`110`	`this.initializeComputerAvatar();`
	`111`	`+ this.pingCRaterEndpoint();`
`111`	`112`	`}`
`112`	`113`
`113`		`- protected startPinging(): void {`
	`114`	`+ private pingCRaterEndpoint(): void {`
`114`	`115`	`this.cRaterPingService.startPinging(this.getItemId());`
`115`	`116`	`}`
`116`	`117`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@`
`34`	`34`	`(ngModelChange)="studentDataChanged()"`
`35`	`35`	`[disabled]="isDisabled"`
`36`	`36`	`cdkTextareaAutosize`
`37`		`- (focus)="onFocus()"`
`38`	`37`	`>`
`39`	`38`	`</textarea>`
`40`	`39`	`</mat-form-field>`
Original file line number	Diff line number	Diff line change
`@@ -107,10 +107,11 @@ export class OpenResponseStudent extends ComponentStudent {`
`107`	`107`	`});`
`108`	`108`	`}`
`109`	`109`	`this.updateAudioAttachments();`
	`110`	`+ this.pingCRaterEndpoint();`
`110`	`111`	`this.broadcastDoneRenderingComponent();`
`111`	`112`	`}`
`112`	`113`
`113`		`- protected onFocus() {`
	`114`	`+ private pingCRaterEndpoint(): void {`
`114`	`115`	`if (this.isCRaterEnabled()) {`
`115`	`116`	`this.cRaterPingService.startPinging(this.getItemId());`
`116`	`117`	`}`