Skip to content

Commit 51c8c0f

Browse files
authored
Fix default merge style for pull requests (#20564) (#20565)
1 parent 8769df1 commit 51c8c0f

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

Diff for: templates/repo/issue/view_content/pull.tmpl

+1
Original file line numberDiff line numberDiff line change
@@ -358,6 +358,7 @@
358358
'emptyCommit': {{.Issue.PullRequest.IsEmpty}},
359359
'pullHeadCommitID': {{.PullHeadCommitID}},
360360
'isPullBranchDeletable': {{.IsPullBranchDeletable}},
361+
'defaultMergeStyle': {{.MergeStyle}},
361362
'defaultDeleteBranchAfterMerge': {{$prUnit.PullRequestsConfig.DefaultDeleteBranchAfterMerge}},
362363
'mergeMessageFieldPlaceHolder': {{$.i18n.Tr "repo.editor.commit_message_desc"}},
363364

Diff for: web_src/js/components/PullRequestMergeForm.vue

+4-1
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,10 @@ export default {
145145
146146
created() {
147147
this.mergeStyleAllowedCount = this.mergeForm.mergeStyles.reduce((v, msd) => v + (msd.allowed ? 1 : 0), 0);
148-
this.switchMergeStyle(this.mergeForm.mergeStyles.find((e) => e.allowed)?.name, !this.mergeForm.canMergeNow);
148+
149+
let mergeStyle = this.mergeForm.mergeStyles.find((e) => e.allowed && e.name === this.mergeForm.defaultMergeStyle)?.name;
150+
if (!mergeStyle) mergeStyle = this.mergeForm.mergeStyles.find((e) => e.allowed)?.name;
151+
this.switchMergeStyle(mergeStyle, !this.mergeForm.canMergeNow);
149152
},
150153
151154
mounted() {

0 commit comments

Comments
 (0)