Skip to content

Migrate widget view events to internal events

Ankit Panchal requested to merge 448283-migrate-widget-view-event into master

What does this MR do and why?

This is 1st part of migrating code review events to internal events.

Type of Events MR Link
view merge request !156609
click on full report merge request TODO
expand merge request TODO

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

  • 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..f01af75e3ec94 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 'MrCodeQualityWidget';
     },
 
     accessibilityWidget() {
-      return this.mr.accessibilityReportPath ? 'MrAccessibilityWidget' : undefined;
+      return 'MrAccessibilityWidget';
     },
 
     widgets() {
  • Open any merge request, it should send track_event API call with test_summary, terraform, code_quality and accessibility events.

Related to #448283

Edited by Ankit Panchal

Merge request reports