Skip to content

Migrate widget full report click events to internal events

What does this MR do and why?

This is 2nd part of migrating code review events to internal events.

Type of Events MR Link
view merge request !156609 (merged)
click on full report merge request !158454 (merged) (We are here)
expand merge request !158561 (merged)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

  1. Apply below patch
diff --git a/app/assets/javascripts/vue_merge_request_widget/components/widget/app.vue b/app/assets/javascripts/vue_merge_request_widget/components/widget/app.vue
index e1d6b22b6f0fb..8c7b99b14bf62 100644
--- a/app/assets/javascripts/vue_merge_request_widget/components/widget/app.vue
+++ b/app/assets/javascripts/vue_merge_request_widget/components/widget/app.vue
@@ -22,19 +22,19 @@ export default {
 
   computed: {
     testReportWidget() {
-      return this.mr.testResultsPath && 'MrTestReportWidget';
+      return 'MrTestReportWidget';
     },
 
     terraformPlansWidget() {
-      return this.mr.terraformReportsPath && 'MrTerraformWidget';
+      return 'MrTerraformWidget';
     },
 
     codeQualityWidget() {
-      return this.mr.codequalityReportsPath ? 'MrCodeQualityWidget' : undefined;
+      return true ? 'MrCodeQualityWidget' : undefined;
     },
 
     accessibilityWidget() {
-      return this.mr.accessibilityReportPath ? 'MrAccessibilityWidget' : undefined;
+      return true ? 'MrAccessibilityWidget' : undefined;
     },
 
     widgets() {
diff --git a/app/assets/javascripts/vue_merge_request_widget/components/widget/widget.vue b/app/assets/javascripts/vue_merge_request_widget/components/widget/widget.vue
index 7e66bb9036dab..71e69d860f105 100644
--- a/app/assets/javascripts/vue_merge_request_widget/components/widget/widget.vue
+++ b/app/assets/javascripts/vue_merge_request_widget/components/widget/widget.vue
@@ -223,6 +223,7 @@ export default {
     }
   },
   async mounted() {
+    this.telemetryHub?.fullReportClicked();
     this.isLoadingCollapsedContent = true;
     this.telemetryHub?.viewed();
  1. Open any merge request in local GDK.
  2. it should send track_event API call with test_summary, terraform, code_quality and accessibility for click full report event. Numbered steps to set up and validate the change are strongly suggested.

Related to #448283 (closed)

Edited by Ankit Panchal

Merge request reports