diff --git a/app/assets/javascripts/vue_merge_request_widget/components/states/ready_to_merge.vue b/app/assets/javascripts/vue_merge_request_widget/components/states/ready_to_merge.vue index 4238198f0fddb3095db1d9c68f0a09042ff7b2bc..59ff7a3cede45de1e280d1a3cb8354730e69f16d 100644 --- a/app/assets/javascripts/vue_merge_request_widget/components/states/ready_to_merge.vue +++ b/app/assets/javascripts/vue_merge_request_widget/components/states/ready_to_merge.vue @@ -415,7 +415,6 @@ export default { const options = { sha: latestSha || this.mr.sha, - commit_message: this.commitMessage, auto_merge_strategy: useAutoMerge ? this.preferredAutoMergeStrategy : undefined, should_remove_source_branch: this.removeSourceBranch === true, squash: this.squashBeforeMerge, @@ -425,10 +424,14 @@ export default { // If users can't alter the squash message (e.g. for 1-commit merge requests), // we shouldn't send the commit message because that would make the backend // do unnecessary work. - if (this.shouldShowSquashBeforeMerge) { + if (this.shouldShowSquashBeforeMerge && this.squashCommitMessageIsTouched) { options.squash_commit_message = this.squashCommitMessage; } + if (this.commitMessageIsTouched) { + options.commit_message = this.commitMessage; + } + this.isMakingRequest = true; this.editCommitMessage = false; @@ -614,6 +617,7 @@ export default { :label="__('Squash commit message')" input-id="squash-message-edit" class="gl-m-0! gl-p-0!" + data-testid="squash-commit-message" @input="setSquashCommitMessage" >