1 files changed, 26 insertions, 23 deletions

diff --git a/app/assets/javascripts/issues/create_merge_request_dropdown.js b/app/assets/javascripts/issues/create_merge_request_dropdown.js
index 8294c018117..edf3789e6dc 100644
--- a/app/assets/javascripts/issues/create_merge_request_dropdown.js
+++ b/app/assets/javascripts/issues/create_merge_request_dropdown.js
@@ -11,6 +11,7 @@ import createFlash from '~/flash';
 import axios from '~/lib/utils/axios_utils';
 import { __, sprintf } from '~/locale';
 import { mergeUrlParams } from '~/lib/utils/url_utility';
+import api from '~/api';
 
 // Todo: Remove this when fixing issue in input_setter plugin
 const InputSetter = { ...ISetter };
@@ -81,10 +82,7 @@ export default class CreateMergeRequestDropdown {
     this.init();
 
     if (isConfidentialIssue()) {
-      this.createMergeRequestButton.setAttribute(
-        'data-dropdown-trigger',
-        '#create-merge-request-dropdown',
-      );
+      this.createMergeRequestButton.dataset.dropdownTrigger = '#create-merge-request-dropdown';
       initConfidentialMergeRequest();
     }
   }
@@ -149,7 +147,7 @@ export default class CreateMergeRequestDropdown {
       });
   }
 
-  createBranch() {
+  createBranch(navigateToBranch = true) {
     this.isCreatingBranch = true;
 
     return axios
@@ -158,7 +156,10 @@ export default class CreateMergeRequestDropdown {
       })
       .then(({ data }) => {
         this.branchCreated = true;
-        window.location.href = data.url;
+
+        if (navigateToBranch) {
+          window.location.href = data.url;
+        }
       })
       .catch(() =>
         createFlash({
@@ -170,23 +171,25 @@ export default class CreateMergeRequestDropdown {
   createMergeRequest() {
     return new Promise(() => {
       this.isCreatingMergeRequest = true;
-      return this.createBranch().then(() => {
-        let path = canCreateConfidentialMergeRequest()
-          ? this.createMrPath.replace(
-              this.projectPath,
-              confidentialMergeRequestState.selectedProject.pathWithNamespace,
-            )
-          : this.createMrPath;
-        path = mergeUrlParams(
-          {
-            'merge_request[target_branch]': this.refInput.value,
-            'merge_request[source_branch]': this.branchInput.value,
-          },
-          path,
-        );
-
-        window.location.href = path;
-      });
+      return this.createBranch(false)
+        .then(() => api.trackRedisHllUserEvent('i_code_review_user_create_mr_from_issue'))
+        .then(() => {
+          let path = canCreateConfidentialMergeRequest()
+            ? this.createMrPath.replace(
+                this.projectPath,
+                confidentialMergeRequestState.selectedProject.pathWithNamespace,
+              )
+            : this.createMrPath;
+          path = mergeUrlParams(
+            {
+              'merge_request[target_branch]': this.refInput.value,
+              'merge_request[source_branch]': this.branchInput.value,
+            },
+            path,
+          );
+
+          window.location.href = path;
+        });
     });
   }