From 7af477e9bea79932ed3d4e336671211615ca122e Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Tue, 16 Jan 2024 22:33:18 -0700 Subject: [PATCH 1/9] Add start review and add to review click events and metrics --- ...est_click_add_to_review_on_changes_tab.yml | 20 ++++++++++++++ ...st_click_add_to_review_on_overview_tab.yml | 20 ++++++++++++++ ...uest_click_start_review_on_changes_tab.yml | 20 ++++++++++++++ ...est_click_start_review_on_overview_tab.yml | 20 ++++++++++++++ ...est_click_add_to_review_on_changes_tab.yml | 26 +++++++++++++++++++ ...st_click_add_to_review_on_overview_tab.yml | 26 +++++++++++++++++++ ...uest_click_start_review_on_changes_tab.yml | 26 +++++++++++++++++++ ...est_click_start_review_on_overview_tab.yml | 26 +++++++++++++++++++ 8 files changed, 184 insertions(+) create mode 100644 config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml create mode 100644 config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml create mode 100644 config/events/code_review_merge_request_click_start_review_on_changes_tab.yml create mode 100644 config/events/code_review_merge_request_click_start_review_on_overview_tab.yml create mode 100644 config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml create mode 100644 config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml create mode 100644 config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml create mode 100644 config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml diff --git a/config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml b/config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml new file mode 100644 index 00000000000000..0eaa4aa34b5413 --- /dev/null +++ b/config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml @@ -0,0 +1,20 @@ +--- +description: User clicks the "Add to review" button on the Merge Request Changes tab +category: InternalEventTracking +action: code_review_merge_request_click_add_to_review_on_changes_tab +identifiers: +- project +- namespace +- user +product_section: dev +product_stage: create +product_group: code_review +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +distributions: +- ce +- ee +tiers: +- free +- premium +- ultimate diff --git a/config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml b/config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml new file mode 100644 index 00000000000000..57b4a245f18c78 --- /dev/null +++ b/config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml @@ -0,0 +1,20 @@ +--- +description: User clicks the "Add to review" button on the Merge Request Overview tab +category: InternalEventTracking +action: code_review_merge_request_click_add_to_review_on_overview_tab +identifiers: +- project +- namespace +- user +product_section: dev +product_stage: create +product_group: code_review +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +distributions: +- ce +- ee +tiers: +- free +- premium +- ultimate diff --git a/config/events/code_review_merge_request_click_start_review_on_changes_tab.yml b/config/events/code_review_merge_request_click_start_review_on_changes_tab.yml new file mode 100644 index 00000000000000..7daf718ea5ed96 --- /dev/null +++ b/config/events/code_review_merge_request_click_start_review_on_changes_tab.yml @@ -0,0 +1,20 @@ +--- +description: User clicks the "Start a review" button on the Merge Request Changes tab +category: InternalEventTracking +action: code_review_merge_request_click_start_review_on_changes_tab +identifiers: +- project +- namespace +- user +product_section: dev +product_stage: create +product_group: code_review +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +distributions: +- ce +- ee +tiers: +- free +- premium +- ultimate diff --git a/config/events/code_review_merge_request_click_start_review_on_overview_tab.yml b/config/events/code_review_merge_request_click_start_review_on_overview_tab.yml new file mode 100644 index 00000000000000..7f39ebd5b2425a --- /dev/null +++ b/config/events/code_review_merge_request_click_start_review_on_overview_tab.yml @@ -0,0 +1,20 @@ +--- +description: User clicks the "Start a review" button on the Merge Request Overview tab +category: InternalEventTracking +action: code_review_merge_request_click_start_review_on_overview_tab +identifiers: +- project +- namespace +- user +product_section: dev +product_stage: create +product_group: code_review +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +distributions: +- ce +- ee +tiers: +- free +- premium +- ultimate diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml b/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml new file mode 100644 index 00000000000000..98e10631dd94dc --- /dev/null +++ b/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab +description: Total count of clicks on the Add to review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: all +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_changes_tab +events: +- name: code_review_merge_request_click_add_to_review_on_changes_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml b/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml new file mode 100644 index 00000000000000..1e5aa129fada03 --- /dev/null +++ b/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab +description: Total count of clicks on the Add to review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: all +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_overview_tab +events: +- name: code_review_merge_request_click_add_to_review_on_overview_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml b/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml new file mode 100644 index 00000000000000..dbc70eb512bea0 --- /dev/null +++ b/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab +description: Total count of clicks on the Start a Review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: all +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_changes_tab +events: +- name: code_review_merge_request_click_start_review_on_changes_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml b/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml new file mode 100644 index 00000000000000..86e35066511c6d --- /dev/null +++ b/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab +description: Total count of clicks on the Start a Review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: all +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_overview_tab +events: +- name: code_review_merge_request_click_start_review_on_overview_tab -- GitLab From f4bc6c2e9403df0cd6f841d90509df8397bcd87a Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Thu, 18 Jan 2024 11:41:02 -0700 Subject: [PATCH 2/9] Add weekly/monthly metrics for starting a review from the overview --- ...k_start_review_on_overview_tab_monthly.yml | 26 +++++++++++++++++++ ...ck_start_review_on_overview_tab_weekly.yml | 26 +++++++++++++++++++ 2 files changed, 52 insertions(+) create mode 100644 config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml create mode 100644 config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml b/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml new file mode 100644 index 00000000000000..f6e06d55ecba85 --- /dev/null +++ b/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly +description: Monthly count of clicks on the Start a Review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 28d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_overview_tab +events: +- name: code_review_merge_request_click_start_review_on_overview_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml b/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml new file mode 100644 index 00000000000000..506db9c0c9c574 --- /dev/null +++ b/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly +description: Weekly count of clicks on the Start a Review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 7d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_overview_tab +events: +- name: code_review_merge_request_click_start_review_on_overview_tab -- GitLab From a28f890324bc56aeb45a646941b31e0da6d5a07a Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Thu, 18 Jan 2024 11:45:42 -0700 Subject: [PATCH 3/9] Add weekly/monthly metrics for starting a review from the changes tab --- ...ck_start_review_on_changes_tab_monthly.yml | 26 +++++++++++++++++++ ...ick_start_review_on_changes_tab_weekly.yml | 26 +++++++++++++++++++ 2 files changed, 52 insertions(+) create mode 100644 config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml create mode 100644 config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml b/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml new file mode 100644 index 00000000000000..8ef2fd9bac6198 --- /dev/null +++ b/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly +description: Monthly count of clicks on the Start a Review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 28d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_changes_tab +events: +- name: code_review_merge_request_click_start_review_on_changes_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml b/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml new file mode 100644 index 00000000000000..2befcd2d411cd3 --- /dev/null +++ b/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly +description: Weekly count of clicks on the Start a Review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 7d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_start_review_on_changes_tab +events: +- name: code_review_merge_request_click_start_review_on_changes_tab -- GitLab From 20720b2c11f180a3740463e21111f254b29d0bdf Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Thu, 18 Jan 2024 11:47:27 -0700 Subject: [PATCH 4/9] Add weekly/monthly metrics for add to review from the overview --- ..._add_to_review_on_overview_tab_monthly.yml | 26 +++++++++++++++++++ ...k_add_to_review_on_overview_tab_weekly.yml | 26 +++++++++++++++++++ 2 files changed, 52 insertions(+) create mode 100644 config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml create mode 100644 config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml b/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml new file mode 100644 index 00000000000000..1e78c041656f21 --- /dev/null +++ b/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly +description: Monthly count of clicks on the Add to review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 28d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_overview_tab +events: +- name: code_review_merge_request_click_add_to_review_on_overview_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml b/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml new file mode 100644 index 00000000000000..755a23c85791fb --- /dev/null +++ b/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly +description: Weekly count of clicks on the Add to review button on the MR Overview tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 7d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_overview_tab +events: +- name: code_review_merge_request_click_add_to_review_on_overview_tab -- GitLab From fd319f4731f566b2bdca7ce6cffe79d52d995f2b Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Thu, 18 Jan 2024 11:48:22 -0700 Subject: [PATCH 5/9] Add weekly/monthly metrics for add to review from the changes tab --- ...k_add_to_review_on_changes_tab_monthly.yml | 26 +++++++++++++++++++ ...ck_add_to_review_on_changes_tab_weekly.yml | 26 +++++++++++++++++++ 2 files changed, 52 insertions(+) create mode 100644 config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml create mode 100644 config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml b/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml new file mode 100644 index 00000000000000..a86e1100999fd5 --- /dev/null +++ b/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly +description: Monthly count of clicks on the Add to review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 28d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_changes_tab +events: +- name: code_review_merge_request_click_add_to_review_on_changes_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml b/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml new file mode 100644 index 00000000000000..b0c6de92718fd6 --- /dev/null +++ b/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml @@ -0,0 +1,26 @@ +--- +key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly +description: Weekly count of clicks on the Add to review button on the MR Changes tab +product_section: dev +product_stage: create +product_group: code_review +performance_indicator_type: [] +value_type: number +status: active +milestone: '16.8' +introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 +time_frame: 7d +data_source: internal_events +data_category: optional +distribution: +- ce +- ee +tier: +- free +- premium +- ultimate +options: + events: + - code_review_merge_request_click_add_to_review_on_changes_tab +events: +- name: code_review_merge_request_click_add_to_review_on_changes_tab -- GitLab From 0a16aab6a4abbb18e90de284be8925294afc65da Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Thu, 18 Jan 2024 21:15:43 -0700 Subject: [PATCH 6/9] Track clicks on the various start/add to review buttons in MRs --- .../javascripts/diffs/components/app.vue | 17 ++++++++++++++++- .../notes/components/comment_form.vue | 7 ++++++- .../javascripts/notes/components/note_form.vue | 2 ++ .../javascripts/notes/components/notes_app.vue | 18 +++++++++++++++++- 4 files changed, 41 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/diffs/components/app.vue b/app/assets/javascripts/diffs/components/app.vue index 978e6731e9d037..64e4004c3dc23d 100644 --- a/app/assets/javascripts/diffs/components/app.vue +++ b/app/assets/javascripts/diffs/components/app.vue @@ -15,6 +15,7 @@ import { MR_COMMITS_PREVIOUS_COMMIT, } from '~/behaviors/shortcuts/keybindings'; import { createAlert } from '~/alert'; +import { InternalEvents } from '~/tracking'; import { isSingleViewStyle } from '~/helpers/diffs_helper'; import { helpPagePath } from '~/helpers/help_page_helper'; import { parseBoolean, handleLocationHash } from '~/lib/utils/common_utils'; @@ -86,7 +87,7 @@ export default { GlSprintf, GlAlert, }, - mixins: [glFeatureFlagsMixin()], + mixins: [glFeatureFlagsMixin(), InternalEvents.mixin()], alerts: { ALERT_OVERFLOW_HIDDEN, ALERT_MERGE_CONFLICT, @@ -456,6 +457,8 @@ export default { notesEventHub.$once('fetchDiffData', this.fetchData); notesEventHub.$on('refetchDiffData', this.refetchDiffData); notesEventHub.$on('fetchedNotesData', this.rereadNoteHash); + notesEventHub.$on('noteFormAddToReview', this.handleReviewTracking); + notesEventHub.$on('noteFormStartReview', this.handleReviewTracking); diffsEventHub.$on('diffFilesModified', this.setDiscussions); diffsEventHub.$on('doneLoadingBatches', this.autoScroll); diffsEventHub.$on(EVT_MR_PREPARED, this.fetchData); @@ -466,6 +469,8 @@ export default { diffsEventHub.$off(EVT_MR_PREPARED, this.fetchData); diffsEventHub.$off('doneLoadingBatches', this.autoScroll); diffsEventHub.$off('diffFilesModified', this.setDiscussions); + notesEventHub.$off('noteFormStartReview', this.handleReviewTracking); + notesEventHub.$off('noteFormAddToReview', this.handleReviewTracking); notesEventHub.$off('fetchedNotesData', this.rereadNoteHash); notesEventHub.$off('refetchDiffData', this.refetchDiffData); notesEventHub.$off('fetchDiffData', this.fetchData); @@ -696,6 +701,16 @@ export default { reloadPage() { window.location.reload(); }, + handleReviewTracking(event) { + const types = { + noteFormStartReview: 'code_review_merge_request_click_start_review_on_changes_tab', + noteFormAddToReview: 'code_review_merge_request_click_add_to_review_on_changes_tab', + }; + + if (this.shouldShow && types[event.name]) { + this.trackEvent(types[event.name]); + } + }, }, howToMergeDocsPath: helpPagePath('user/project/merge_requests/reviews/index.md', { anchor: 'checkout-merge-requests-locally-through-the-head-ref', diff --git a/app/assets/javascripts/notes/components/comment_form.vue b/app/assets/javascripts/notes/components/comment_form.vue index 87b55b19c08afc..17eded3bec0445 100644 --- a/app/assets/javascripts/notes/components/comment_form.vue +++ b/app/assets/javascripts/notes/components/comment_form.vue @@ -12,6 +12,7 @@ import { slugifyWithUnderscore, } from '~/lib/utils/text_utility'; import { sprintf } from '~/locale'; +import { InternalEvents } from '~/tracking'; import { badgeState } from '~/merge_requests/components/merge_request_header.vue'; import MarkdownEditor from '~/vue_shared/components/markdown/markdown_editor.vue'; import TimelineEntryItem from '~/vue_shared/components/notes/timeline_entry_item.vue'; @@ -48,7 +49,7 @@ export default { directives: { GlTooltip: GlTooltipDirective, }, - mixins: [issuableStateMixin], + mixins: [issuableStateMixin, InternalEvents.mixin()], props: { noteableType: { type: String, @@ -253,6 +254,10 @@ export default { this.isSubmitting = true; + if (isDraft) { + eventHub.$emit('noteFormAddToReview', { name: 'noteFormAddToReview' }); + } + trackSavedUsingEditor( this.$refs.markdownEditor.isContentEditorActive, `${this.noteableType}_${this.noteType}`, diff --git a/app/assets/javascripts/notes/components/note_form.vue b/app/assets/javascripts/notes/components/note_form.vue index 77ce5ea5910fe9..135d595aae59f6 100644 --- a/app/assets/javascripts/notes/components/note_form.vue +++ b/app/assets/javascripts/notes/components/note_form.vue @@ -320,12 +320,14 @@ export default { ); }, handleAddToReview() { + const clickType = this.hasDrafts ? 'noteFormAddToReview' : 'noteFormStartReview'; // check if draft should resolve thread const shouldResolve = (this.discussionResolved && !this.isUnresolving) || (!this.discussionResolved && this.isResolving); this.isSubmitting = true; + eventHub.$emit(clickType, { name: clickType }); this.$emit( 'handleFormUpdateAddToReview', this.updatedNoteBody, diff --git a/app/assets/javascripts/notes/components/notes_app.vue b/app/assets/javascripts/notes/components/notes_app.vue index 86f93ee425e2fb..40cc6c8847c0e5 100644 --- a/app/assets/javascripts/notes/components/notes_app.vue +++ b/app/assets/javascripts/notes/components/notes_app.vue @@ -2,6 +2,7 @@ // eslint-disable-next-line no-restricted-imports import { mapGetters, mapActions } from 'vuex'; import { v4 as uuidv4 } from 'uuid'; +import { InternalEvents } from '~/tracking'; import highlightCurrentUser from '~/behaviors/markdown/highlight_current_user'; import TimelineEntryItem from '~/vue_shared/components/notes/timeline_entry_item.vue'; import OrderedLayout from '~/vue_shared/components/ordered_layout.vue'; @@ -39,7 +40,7 @@ export default { TimelineEntryItem, AiSummary: () => import('ee_component/notes/components/ai_summary.vue'), }, - mixins: [glFeatureFlagsMixin()], + mixins: [glFeatureFlagsMixin(), InternalEvents.mixin()], provide() { return { summarizeClientSubscriptionId: uuidv4(), @@ -165,6 +166,9 @@ export default { }); } + eventHub.$on('noteFormAddToReview', this.handleReviewTracking); + eventHub.$on('noteFormStartReview', this.handleReviewTracking); + window.addEventListener('hashchange', this.handleHashChanged); eventHub.$on('notesApp.updateIssuableConfidentiality', this.setConfidentiality); @@ -177,6 +181,8 @@ export default { beforeDestroy() { window.removeEventListener('hashchange', this.handleHashChanged); eventHub.$off('notesApp.updateIssuableConfidentiality', this.setConfidentiality); + eventHub.$off('noteFormStartReview', this.handleReviewTracking); + eventHub.$off('noteFormAddToReview', this.handleReviewTracking); }, methods: { ...mapActions([ @@ -222,6 +228,16 @@ export default { setAiLoading(loading) { this.aiLoading = loading; }, + handleReviewTracking(event) { + const types = { + noteFormStartReview: 'code_review_merge_request_click_start_review_on_overview_tab', + noteFormAddToReview: 'code_review_merge_request_click_add_to_review_on_overview_tab', + }; + + if (this.shouldShow && window.mrTabs && types[event.name]) { + this.trackEvent(types[event.name]); + } + }, }, systemNote: constants.SYSTEM_NOTE, }; -- GitLab From 7a7fe4fcb848c4c60639bc579c267dc6f6ab4597 Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Fri, 19 Jan 2024 01:05:44 -0700 Subject: [PATCH 7/9] Test that UI interactions trigger the appropriate events --- .../notes/components/comment_form_spec.js | 19 ++++++- .../notes/components/note_form_spec.js | 51 +++++++++++++++---- 2 files changed, 58 insertions(+), 12 deletions(-) diff --git a/spec/frontend/notes/components/comment_form_spec.js b/spec/frontend/notes/components/comment_form_spec.js index 8f761476c7c99e..500032eac26cb2 100644 --- a/spec/frontend/notes/components/comment_form_spec.js +++ b/spec/frontend/notes/components/comment_form_spec.js @@ -401,11 +401,28 @@ describe('issue_comment_form component', () => { let store; beforeEach(() => { - store = createStore(); + store = createStore({ + actions: { + saveNote: jest.fn().mockResolvedValue(), + }, + }); store.registerModule('batchComments', batchComments()); store.state.batchComments.drafts = [{ note: 'A' }]; }); + it('sends the event to indicate that a new draft comment has been added', () => { + const note = 'some note text which enables actually adding a draft note'; + + jest.spyOn(eventHub, '$emit'); + mountComponent({ mountFunction: mount, initialData: { note }, store }); + + findAddToReviewButton().trigger('click'); + + expect(eventHub.$emit).toHaveBeenCalledWith('noteFormAddToReview', { + name: 'noteFormAddToReview', + }); + }); + it('should save note draft when cmd+enter is pressed', async () => { mountComponent({ mountFunction: mount, store }); jest.spyOn(wrapper.vm, 'handleSaveDraft'); diff --git a/spec/frontend/notes/components/note_form_spec.js b/spec/frontend/notes/components/note_form_spec.js index e2072ebd04d178..dad36a290b1dc2 100644 --- a/spec/frontend/notes/components/note_form_spec.js +++ b/spec/frontend/notes/components/note_form_spec.js @@ -7,6 +7,7 @@ import MarkdownField from '~/vue_shared/components/markdown/field.vue'; import CommentFieldLayout from '~/notes/components/comment_field_layout.vue'; import { AT_WHO_ACTIVE_CLASS } from '~/gfm_auto_complete'; import eventHub from '~/environments/event_hub'; +import notesEventHub from '~/notes/event_hub'; import { mountExtended } from 'helpers/vue_test_utils_helper'; import { mockTracking } from 'helpers/tracking_helper'; import { noteableDataMock, notesDataMock, discussionMock, note } from '../mock_data'; @@ -16,6 +17,7 @@ jest.mock('~/lib/utils/autosave'); describe('issue_note_form component', () => { let store; let wrapper; + let textarea; let props; let trackingSpy; @@ -39,10 +41,13 @@ describe('issue_note_form component', () => { }, }, }); + + textarea = wrapper.find('textarea'); }; const findCancelButton = () => wrapper.findByTestId('cancel'); const findCancelCommentButton = () => wrapper.findByTestId('cancelBatchCommentsEnabled'); + const findAddToStartReviewButton = () => wrapper.findByTestId('start-review-button'); const findMarkdownField = () => wrapper.findComponent(MarkdownField); beforeEach(() => { @@ -112,14 +117,10 @@ describe('issue_note_form component', () => { }); it('should render text area with placeholder', () => { - const textarea = wrapper.find('textarea'); - expect(textarea.attributes('placeholder')).toBe('Write a comment or drag your files here…'); }); it('should set data-supports-quick-actions to enable autocomplete', () => { - const textarea = wrapper.find('textarea'); - expect(textarea.attributes('data-supports-quick-actions')).toBe('true'); }); @@ -147,10 +148,7 @@ describe('issue_note_form component', () => { }); describe('keyboard events', () => { - let textarea; - beforeEach(() => { - textarea = wrapper.find('textarea'); textarea.setValue('Foo'); }); @@ -213,7 +211,6 @@ describe('issue_note_form component', () => { it('should be possible to update the note', () => { createComponentWrapper(); - const textarea = wrapper.find('textarea'); textarea.setValue('Foo'); const saveButton = wrapper.find('.js-vue-issue-save'); saveButton.vm.$emit('click'); @@ -224,7 +221,6 @@ describe('issue_note_form component', () => { it('tracks event when save button is clicked', () => { createComponentWrapper(); - const textarea = wrapper.find('textarea'); textarea.setValue('Foo'); const saveButton = wrapper.find('.js-vue-issue-save'); saveButton.vm.$emit('click'); @@ -313,8 +309,6 @@ describe('issue_note_form component', () => { describe('on enter', () => { it('should start review or add to review when cmd+enter is pressed', async () => { - const textarea = wrapper.find('textarea'); - textarea.setValue('Foo'); textarea.trigger('keydown.enter', { metaKey: true }); @@ -325,5 +319,40 @@ describe('issue_note_form component', () => { ]); }); }); + + describe('when adding a draft comment', () => { + beforeEach(() => { + jest.spyOn(notesEventHub, '$emit'); + }); + + it('sends the event to indicate that a draft has been added to the review', () => { + store.state.batchComments.drafts = [{ note: 'A' }]; + createComponentWrapper({ + isDraft: true, + noteId: '', + discussion: { ...discussionMock, for_commit: false }, + }); + + findAddToStartReviewButton().trigger('click'); + + expect(notesEventHub.$emit).toHaveBeenCalledWith('noteFormAddToReview', { + name: 'noteFormAddToReview', + }); + }); + + it('sends the event to indicate that a review has been started with the new draft', () => { + createComponentWrapper({ + isDraft: true, + noteId: '', + discussion: { ...discussionMock, for_commit: false }, + }); + + findAddToStartReviewButton().trigger('click'); + + expect(notesEventHub.$emit).toHaveBeenCalledWith('noteFormStartReview', { + name: 'noteFormStartReview', + }); + }); + }); }); }); -- GitLab From f9d2df7f5bfbd5f21081d1171eb20c6ac49d0824 Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Fri, 19 Jan 2024 01:50:01 -0700 Subject: [PATCH 8/9] Test that draft comment events trigger the correct tracking events --- spec/frontend/diffs/components/app_spec.js | 36 ++++++++++++++++++ .../notes/components/notes_app_spec.js | 37 +++++++++++++++++++ 2 files changed, 73 insertions(+) diff --git a/spec/frontend/diffs/components/app_spec.js b/spec/frontend/diffs/components/app_spec.js index 4676f56c47ee9a..6b80b4ee1c100d 100644 --- a/spec/frontend/diffs/components/app_spec.js +++ b/spec/frontend/diffs/components/app_spec.js @@ -8,6 +8,7 @@ import Vuex from 'vuex'; import getMRCodequalityAndSecurityReports from '~/diffs/components/graphql/get_mr_codequality_and_security_reports.query.graphql'; import createMockApollo from 'helpers/mock_apollo_helper'; import setWindowLocation from 'helpers/set_window_location_helper'; +import { mockTracking } from 'helpers/tracking_helper'; import { TEST_HOST } from 'spec/test_constants'; import App from '~/diffs/components/app.vue'; @@ -22,6 +23,7 @@ import CollapsedFilesWarning from '~/diffs/components/collapsed_files_warning.vu import HiddenFilesWarning from '~/diffs/components/hidden_files_warning.vue'; import eventHub from '~/diffs/event_hub'; +import notesEventHub from '~/notes/event_hub'; import { EVT_DISCUSSIONS_ASSIGNED } from '~/diffs/constants'; import axios from '~/lib/utils/axios_utils'; @@ -892,4 +894,38 @@ describe('diffs/components/app', () => { expect(wrapper.findComponent(GlLoadingIcon).exists()).toBe(true); }); }); + + describe('draft comments', () => { + let trackingSpy; + + beforeEach(() => { + trackingSpy = mockTracking(undefined, window.document, jest.spyOn); + }); + + describe('when adding a new comment to an existing review', () => { + it('sends the correct tracking event', () => { + createComponent({ shouldShow: true }); + notesEventHub.$emit('noteFormAddToReview', { name: 'noteFormAddToReview' }); + + expect(trackingSpy).toHaveBeenCalledWith( + undefined, + 'code_review_merge_request_click_add_to_review_on_changes_tab', + expect.any(Object), + ); + }); + }); + + describe('when adding a comment to a new review', () => { + it('sends the correct tracking event', () => { + createComponent({ shouldShow: true }); + notesEventHub.$emit('noteFormStartReview', { name: 'noteFormStartReview' }); + + expect(trackingSpy).toHaveBeenCalledWith( + undefined, + 'code_review_merge_request_click_start_review_on_changes_tab', + expect.any(Object), + ); + }); + }); + }); }); diff --git a/spec/frontend/notes/components/notes_app_spec.js b/spec/frontend/notes/components/notes_app_spec.js index d49ab0d71dbdfa..003f12fce02067 100644 --- a/spec/frontend/notes/components/notes_app_spec.js +++ b/spec/frontend/notes/components/notes_app_spec.js @@ -3,6 +3,7 @@ import AxiosMockAdapter from 'axios-mock-adapter'; import $ from 'jquery'; import { nextTick } from 'vue'; import setWindowLocation from 'helpers/set_window_location_helper'; +import { mockTracking } from 'helpers/tracking_helper'; import waitForPromises from 'helpers/wait_for_promises'; import DraftNote from '~/batch_comments/components/draft_note.vue'; import batchComments from '~/batch_comments/stores/modules/batch_comments'; @@ -10,6 +11,7 @@ import axios from '~/lib/utils/axios_utils'; import { HTTP_STATUS_OK } from '~/lib/utils/http_status'; import { getLocationHash } from '~/lib/utils/url_utility'; import * as urlUtility from '~/lib/utils/url_utility'; +import notesEventHub from '~/notes/event_hub'; import CommentForm from '~/notes/components/comment_form.vue'; import NotesApp from '~/notes/components/notes_app.vue'; import NotesActivityHeader from '~/notes/components/notes_activity_header.vue'; @@ -453,4 +455,39 @@ describe('note_app', () => { }); }); }); + + describe('draft comments', () => { + let trackingSpy; + + beforeEach(() => { + window.mrTabs = { eventHub: notesEventHub }; + axiosMock.onAny().reply(mockData.getIndividualNoteResponse); + trackingSpy = mockTracking(undefined, window.document, jest.spyOn); + wrapper = mountComponent(); + }); + + describe('when adding a new comment to an existing review', () => { + it('sends the correct tracking event', () => { + notesEventHub.$emit('noteFormAddToReview', { name: 'noteFormAddToReview' }); + + expect(trackingSpy).toHaveBeenCalledWith( + undefined, + 'code_review_merge_request_click_add_to_review_on_overview_tab', + expect.any(Object), + ); + }); + }); + + describe('when adding a comment to a new review', () => { + it('sends the correct tracking event', () => { + notesEventHub.$emit('noteFormStartReview', { name: 'noteFormStartReview' }); + + expect(trackingSpy).toHaveBeenCalledWith( + undefined, + 'code_review_merge_request_click_start_review_on_overview_tab', + expect.any(Object), + ); + }); + }); + }); }); -- GitLab From 3cbaab1e27cb3a25370ba9f372858c39534c846f Mon Sep 17 00:00:00 2001 From: Thomas Randolph Date: Fri, 19 Jan 2024 09:42:03 -0700 Subject: [PATCH 9/9] Remove `code_review_` prefixes and update milestones --- app/assets/javascripts/diffs/components/app.vue | 4 ++-- app/assets/javascripts/notes/components/notes_app.vue | 4 ++-- ... merge_request_click_add_to_review_on_changes_tab.yml} | 4 ++-- ...merge_request_click_add_to_review_on_overview_tab.yml} | 4 ++-- ...> merge_request_click_start_review_on_changes_tab.yml} | 4 ++-- ... merge_request_click_start_review_on_overview_tab.yml} | 4 ++-- ...equest_click_add_to_review_on_changes_tab_monthly.yml} | 8 ++++---- ...quest_click_add_to_review_on_overview_tab_monthly.yml} | 8 ++++---- ...request_click_start_review_on_changes_tab_monthly.yml} | 8 ++++---- ...equest_click_start_review_on_overview_tab_monthly.yml} | 8 ++++---- ...request_click_add_to_review_on_changes_tab_weekly.yml} | 8 ++++---- ...equest_click_add_to_review_on_overview_tab_weekly.yml} | 8 ++++---- ..._request_click_start_review_on_changes_tab_weekly.yml} | 8 ++++---- ...request_click_start_review_on_overview_tab_weekly.yml} | 8 ++++---- ..._merge_request_click_add_to_review_on_changes_tab.yml} | 8 ++++---- ...merge_request_click_add_to_review_on_overview_tab.yml} | 8 ++++---- ...l_merge_request_click_start_review_on_changes_tab.yml} | 8 ++++---- ..._merge_request_click_start_review_on_overview_tab.yml} | 8 ++++---- spec/frontend/diffs/components/app_spec.js | 4 ++-- spec/frontend/notes/components/notes_app_spec.js | 4 ++-- 20 files changed, 64 insertions(+), 64 deletions(-) rename config/events/{code_review_merge_request_click_add_to_review_on_changes_tab.yml => merge_request_click_add_to_review_on_changes_tab.yml} (81%) rename config/events/{code_review_merge_request_click_add_to_review_on_overview_tab.yml => merge_request_click_add_to_review_on_overview_tab.yml} (80%) rename config/events/{code_review_merge_request_click_start_review_on_changes_tab.yml => merge_request_click_start_review_on_changes_tab.yml} (81%) rename config/events/{code_review_merge_request_click_start_review_on_overview_tab.yml => merge_request_click_start_review_on_overview_tab.yml} (81%) rename config/metrics/counts_28d/{count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml => count_total_merge_request_click_add_to_review_on_changes_tab_monthly.yml} (64%) rename config/metrics/counts_28d/{count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml => count_total_merge_request_click_add_to_review_on_overview_tab_monthly.yml} (64%) rename config/metrics/counts_28d/{count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml => count_total_merge_request_click_start_review_on_changes_tab_monthly.yml} (65%) rename config/metrics/counts_28d/{count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml => count_total_merge_request_click_start_review_on_overview_tab_monthly.yml} (64%) rename config/metrics/counts_7d/{count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml => count_total_merge_request_click_add_to_review_on_changes_tab_weekly.yml} (64%) rename config/metrics/counts_7d/{count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml => count_total_merge_request_click_add_to_review_on_overview_tab_weekly.yml} (64%) rename config/metrics/counts_7d/{count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml => count_total_merge_request_click_start_review_on_changes_tab_weekly.yml} (65%) rename config/metrics/counts_7d/{count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml => count_total_merge_request_click_start_review_on_overview_tab_weekly.yml} (64%) rename config/metrics/counts_all/{count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml => count_total_merge_request_click_add_to_review_on_changes_tab.yml} (65%) rename config/metrics/counts_all/{count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml => count_total_merge_request_click_add_to_review_on_overview_tab.yml} (65%) rename config/metrics/counts_all/{count_total_code_review_merge_request_click_start_review_on_changes_tab.yml => count_total_merge_request_click_start_review_on_changes_tab.yml} (65%) rename config/metrics/counts_all/{count_total_code_review_merge_request_click_start_review_on_overview_tab.yml => count_total_merge_request_click_start_review_on_overview_tab.yml} (65%) diff --git a/app/assets/javascripts/diffs/components/app.vue b/app/assets/javascripts/diffs/components/app.vue index 64e4004c3dc23d..787dd0e8e66e20 100644 --- a/app/assets/javascripts/diffs/components/app.vue +++ b/app/assets/javascripts/diffs/components/app.vue @@ -703,8 +703,8 @@ export default { }, handleReviewTracking(event) { const types = { - noteFormStartReview: 'code_review_merge_request_click_start_review_on_changes_tab', - noteFormAddToReview: 'code_review_merge_request_click_add_to_review_on_changes_tab', + noteFormStartReview: 'merge_request_click_start_review_on_changes_tab', + noteFormAddToReview: 'merge_request_click_add_to_review_on_changes_tab', }; if (this.shouldShow && types[event.name]) { diff --git a/app/assets/javascripts/notes/components/notes_app.vue b/app/assets/javascripts/notes/components/notes_app.vue index 40cc6c8847c0e5..eb6764a7937813 100644 --- a/app/assets/javascripts/notes/components/notes_app.vue +++ b/app/assets/javascripts/notes/components/notes_app.vue @@ -230,8 +230,8 @@ export default { }, handleReviewTracking(event) { const types = { - noteFormStartReview: 'code_review_merge_request_click_start_review_on_overview_tab', - noteFormAddToReview: 'code_review_merge_request_click_add_to_review_on_overview_tab', + noteFormStartReview: 'merge_request_click_start_review_on_overview_tab', + noteFormAddToReview: 'merge_request_click_add_to_review_on_overview_tab', }; if (this.shouldShow && window.mrTabs && types[event.name]) { diff --git a/config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml b/config/events/merge_request_click_add_to_review_on_changes_tab.yml similarity index 81% rename from config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml rename to config/events/merge_request_click_add_to_review_on_changes_tab.yml index 0eaa4aa34b5413..5d504b617a0a49 100644 --- a/config/events/code_review_merge_request_click_add_to_review_on_changes_tab.yml +++ b/config/events/merge_request_click_add_to_review_on_changes_tab.yml @@ -1,7 +1,7 @@ --- description: User clicks the "Add to review" button on the Merge Request Changes tab category: InternalEventTracking -action: code_review_merge_request_click_add_to_review_on_changes_tab +action: merge_request_click_add_to_review_on_changes_tab identifiers: - project - namespace @@ -9,7 +9,7 @@ identifiers: product_section: dev product_stage: create product_group: code_review -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 distributions: - ce diff --git a/config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml b/config/events/merge_request_click_add_to_review_on_overview_tab.yml similarity index 80% rename from config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml rename to config/events/merge_request_click_add_to_review_on_overview_tab.yml index 57b4a245f18c78..0001aada7b47c0 100644 --- a/config/events/code_review_merge_request_click_add_to_review_on_overview_tab.yml +++ b/config/events/merge_request_click_add_to_review_on_overview_tab.yml @@ -1,7 +1,7 @@ --- description: User clicks the "Add to review" button on the Merge Request Overview tab category: InternalEventTracking -action: code_review_merge_request_click_add_to_review_on_overview_tab +action: merge_request_click_add_to_review_on_overview_tab identifiers: - project - namespace @@ -9,7 +9,7 @@ identifiers: product_section: dev product_stage: create product_group: code_review -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 distributions: - ce diff --git a/config/events/code_review_merge_request_click_start_review_on_changes_tab.yml b/config/events/merge_request_click_start_review_on_changes_tab.yml similarity index 81% rename from config/events/code_review_merge_request_click_start_review_on_changes_tab.yml rename to config/events/merge_request_click_start_review_on_changes_tab.yml index 7daf718ea5ed96..a1cd6c0bda3bba 100644 --- a/config/events/code_review_merge_request_click_start_review_on_changes_tab.yml +++ b/config/events/merge_request_click_start_review_on_changes_tab.yml @@ -1,7 +1,7 @@ --- description: User clicks the "Start a review" button on the Merge Request Changes tab category: InternalEventTracking -action: code_review_merge_request_click_start_review_on_changes_tab +action: merge_request_click_start_review_on_changes_tab identifiers: - project - namespace @@ -9,7 +9,7 @@ identifiers: product_section: dev product_stage: create product_group: code_review -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 distributions: - ce diff --git a/config/events/code_review_merge_request_click_start_review_on_overview_tab.yml b/config/events/merge_request_click_start_review_on_overview_tab.yml similarity index 81% rename from config/events/code_review_merge_request_click_start_review_on_overview_tab.yml rename to config/events/merge_request_click_start_review_on_overview_tab.yml index 7f39ebd5b2425a..ec7be2a6c1ee44 100644 --- a/config/events/code_review_merge_request_click_start_review_on_overview_tab.yml +++ b/config/events/merge_request_click_start_review_on_overview_tab.yml @@ -1,7 +1,7 @@ --- description: User clicks the "Start a review" button on the Merge Request Overview tab category: InternalEventTracking -action: code_review_merge_request_click_start_review_on_overview_tab +action: merge_request_click_start_review_on_overview_tab identifiers: - project - namespace @@ -9,7 +9,7 @@ identifiers: product_section: dev product_stage: create product_group: code_review -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 distributions: - ce diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml b/config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_changes_tab_monthly.yml similarity index 64% rename from config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml rename to config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_changes_tab_monthly.yml index a86e1100999fd5..160258d006e7dc 100644 --- a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly.yml +++ b/config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_changes_tab_monthly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab_monthly +key_path: counts.count_total_merge_request_click_add_to_review_on_changes_tab_monthly description: Monthly count of clicks on the Add to review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 28d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_changes_tab + - merge_request_click_add_to_review_on_changes_tab events: -- name: code_review_merge_request_click_add_to_review_on_changes_tab +- name: merge_request_click_add_to_review_on_changes_tab diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml b/config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_overview_tab_monthly.yml similarity index 64% rename from config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml rename to config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_overview_tab_monthly.yml index 1e78c041656f21..8f6942d62e7666 100644 --- a/config/metrics/counts_28d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly.yml +++ b/config/metrics/counts_28d/count_total_merge_request_click_add_to_review_on_overview_tab_monthly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab_monthly +key_path: counts.count_total_merge_request_click_add_to_review_on_overview_tab_monthly description: Monthly count of clicks on the Add to review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 28d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_overview_tab + - merge_request_click_add_to_review_on_overview_tab events: -- name: code_review_merge_request_click_add_to_review_on_overview_tab +- name: merge_request_click_add_to_review_on_overview_tab diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml b/config/metrics/counts_28d/count_total_merge_request_click_start_review_on_changes_tab_monthly.yml similarity index 65% rename from config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml rename to config/metrics/counts_28d/count_total_merge_request_click_start_review_on_changes_tab_monthly.yml index 8ef2fd9bac6198..8b76940597963f 100644 --- a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly.yml +++ b/config/metrics/counts_28d/count_total_merge_request_click_start_review_on_changes_tab_monthly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab_monthly +key_path: counts.count_total_merge_request_click_start_review_on_changes_tab_monthly description: Monthly count of clicks on the Start a Review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 28d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_changes_tab + - merge_request_click_start_review_on_changes_tab events: -- name: code_review_merge_request_click_start_review_on_changes_tab +- name: merge_request_click_start_review_on_changes_tab diff --git a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml b/config/metrics/counts_28d/count_total_merge_request_click_start_review_on_overview_tab_monthly.yml similarity index 64% rename from config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml rename to config/metrics/counts_28d/count_total_merge_request_click_start_review_on_overview_tab_monthly.yml index f6e06d55ecba85..203cd468f3da38 100644 --- a/config/metrics/counts_28d/count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly.yml +++ b/config/metrics/counts_28d/count_total_merge_request_click_start_review_on_overview_tab_monthly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab_monthly +key_path: counts.count_total_merge_request_click_start_review_on_overview_tab_monthly description: Monthly count of clicks on the Start a Review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 28d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_overview_tab + - merge_request_click_start_review_on_overview_tab events: -- name: code_review_merge_request_click_start_review_on_overview_tab +- name: merge_request_click_start_review_on_overview_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml b/config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_changes_tab_weekly.yml similarity index 64% rename from config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml rename to config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_changes_tab_weekly.yml index b0c6de92718fd6..d764a377dc5dc2 100644 --- a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly.yml +++ b/config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_changes_tab_weekly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab_weekly +key_path: counts.count_total_merge_request_click_add_to_review_on_changes_tab_weekly description: Weekly count of clicks on the Add to review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 7d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_changes_tab + - merge_request_click_add_to_review_on_changes_tab events: -- name: code_review_merge_request_click_add_to_review_on_changes_tab +- name: merge_request_click_add_to_review_on_changes_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml b/config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_overview_tab_weekly.yml similarity index 64% rename from config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml rename to config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_overview_tab_weekly.yml index 755a23c85791fb..c11ac48986e5ec 100644 --- a/config/metrics/counts_7d/count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly.yml +++ b/config/metrics/counts_7d/count_total_merge_request_click_add_to_review_on_overview_tab_weekly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab_weekly +key_path: counts.count_total_merge_request_click_add_to_review_on_overview_tab_weekly description: Weekly count of clicks on the Add to review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 7d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_overview_tab + - merge_request_click_add_to_review_on_overview_tab events: -- name: code_review_merge_request_click_add_to_review_on_overview_tab +- name: merge_request_click_add_to_review_on_overview_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml b/config/metrics/counts_7d/count_total_merge_request_click_start_review_on_changes_tab_weekly.yml similarity index 65% rename from config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml rename to config/metrics/counts_7d/count_total_merge_request_click_start_review_on_changes_tab_weekly.yml index 2befcd2d411cd3..0fdf6762da1a06 100644 --- a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly.yml +++ b/config/metrics/counts_7d/count_total_merge_request_click_start_review_on_changes_tab_weekly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab_weekly +key_path: counts.count_total_merge_request_click_start_review_on_changes_tab_weekly description: Weekly count of clicks on the Start a Review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 7d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_changes_tab + - merge_request_click_start_review_on_changes_tab events: -- name: code_review_merge_request_click_start_review_on_changes_tab +- name: merge_request_click_start_review_on_changes_tab diff --git a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml b/config/metrics/counts_7d/count_total_merge_request_click_start_review_on_overview_tab_weekly.yml similarity index 64% rename from config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml rename to config/metrics/counts_7d/count_total_merge_request_click_start_review_on_overview_tab_weekly.yml index 506db9c0c9c574..4ff3f9b090c05e 100644 --- a/config/metrics/counts_7d/count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly.yml +++ b/config/metrics/counts_7d/count_total_merge_request_click_start_review_on_overview_tab_weekly.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab_weekly +key_path: counts.count_total_merge_request_click_start_review_on_overview_tab_weekly description: Weekly count of clicks on the Start a Review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: 7d data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_overview_tab + - merge_request_click_start_review_on_overview_tab events: -- name: code_review_merge_request_click_start_review_on_overview_tab +- name: merge_request_click_start_review_on_overview_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml b/config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_changes_tab.yml similarity index 65% rename from config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml rename to config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_changes_tab.yml index 98e10631dd94dc..0383f854a4ec9a 100644 --- a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_changes_tab.yml +++ b/config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_changes_tab.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_changes_tab +key_path: counts.count_total_merge_request_click_add_to_review_on_changes_tab description: Total count of clicks on the Add to review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: all data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_changes_tab + - merge_request_click_add_to_review_on_changes_tab events: -- name: code_review_merge_request_click_add_to_review_on_changes_tab +- name: merge_request_click_add_to_review_on_changes_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml b/config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_overview_tab.yml similarity index 65% rename from config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml rename to config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_overview_tab.yml index 1e5aa129fada03..e9ed2d5c155c5a 100644 --- a/config/metrics/counts_all/count_total_code_review_merge_request_click_add_to_review_on_overview_tab.yml +++ b/config/metrics/counts_all/count_total_merge_request_click_add_to_review_on_overview_tab.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_add_to_review_on_overview_tab +key_path: counts.count_total_merge_request_click_add_to_review_on_overview_tab description: Total count of clicks on the Add to review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: all data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_add_to_review_on_overview_tab + - merge_request_click_add_to_review_on_overview_tab events: -- name: code_review_merge_request_click_add_to_review_on_overview_tab +- name: merge_request_click_add_to_review_on_overview_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml b/config/metrics/counts_all/count_total_merge_request_click_start_review_on_changes_tab.yml similarity index 65% rename from config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml rename to config/metrics/counts_all/count_total_merge_request_click_start_review_on_changes_tab.yml index dbc70eb512bea0..bac1dbbc1cb9fc 100644 --- a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_changes_tab.yml +++ b/config/metrics/counts_all/count_total_merge_request_click_start_review_on_changes_tab.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_changes_tab +key_path: counts.count_total_merge_request_click_start_review_on_changes_tab description: Total count of clicks on the Start a Review button on the MR Changes tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: all data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_changes_tab + - merge_request_click_start_review_on_changes_tab events: -- name: code_review_merge_request_click_start_review_on_changes_tab +- name: merge_request_click_start_review_on_changes_tab diff --git a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml b/config/metrics/counts_all/count_total_merge_request_click_start_review_on_overview_tab.yml similarity index 65% rename from config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml rename to config/metrics/counts_all/count_total_merge_request_click_start_review_on_overview_tab.yml index 86e35066511c6d..7ee5058c7ca972 100644 --- a/config/metrics/counts_all/count_total_code_review_merge_request_click_start_review_on_overview_tab.yml +++ b/config/metrics/counts_all/count_total_merge_request_click_start_review_on_overview_tab.yml @@ -1,5 +1,5 @@ --- -key_path: counts.count_total_code_review_merge_request_click_start_review_on_overview_tab +key_path: counts.count_total_merge_request_click_start_review_on_overview_tab description: Total count of clicks on the Start a Review button on the MR Overview tab product_section: dev product_stage: create @@ -7,7 +7,7 @@ product_group: code_review performance_indicator_type: [] value_type: number status: active -milestone: '16.8' +milestone: '16.9' introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/141911 time_frame: all data_source: internal_events @@ -21,6 +21,6 @@ tier: - ultimate options: events: - - code_review_merge_request_click_start_review_on_overview_tab + - merge_request_click_start_review_on_overview_tab events: -- name: code_review_merge_request_click_start_review_on_overview_tab +- name: merge_request_click_start_review_on_overview_tab diff --git a/spec/frontend/diffs/components/app_spec.js b/spec/frontend/diffs/components/app_spec.js index 6b80b4ee1c100d..99b3f03ee32005 100644 --- a/spec/frontend/diffs/components/app_spec.js +++ b/spec/frontend/diffs/components/app_spec.js @@ -909,7 +909,7 @@ describe('diffs/components/app', () => { expect(trackingSpy).toHaveBeenCalledWith( undefined, - 'code_review_merge_request_click_add_to_review_on_changes_tab', + 'merge_request_click_add_to_review_on_changes_tab', expect.any(Object), ); }); @@ -922,7 +922,7 @@ describe('diffs/components/app', () => { expect(trackingSpy).toHaveBeenCalledWith( undefined, - 'code_review_merge_request_click_start_review_on_changes_tab', + 'merge_request_click_start_review_on_changes_tab', expect.any(Object), ); }); diff --git a/spec/frontend/notes/components/notes_app_spec.js b/spec/frontend/notes/components/notes_app_spec.js index 003f12fce02067..e278d53b55af83 100644 --- a/spec/frontend/notes/components/notes_app_spec.js +++ b/spec/frontend/notes/components/notes_app_spec.js @@ -472,7 +472,7 @@ describe('note_app', () => { expect(trackingSpy).toHaveBeenCalledWith( undefined, - 'code_review_merge_request_click_add_to_review_on_overview_tab', + 'merge_request_click_add_to_review_on_overview_tab', expect.any(Object), ); }); @@ -484,7 +484,7 @@ describe('note_app', () => { expect(trackingSpy).toHaveBeenCalledWith( undefined, - 'code_review_merge_request_click_start_review_on_overview_tab', + 'merge_request_click_start_review_on_overview_tab', expect.any(Object), ); }); -- GitLab