Skip to content

Commit 7c3ba8b

Browse files
authored
Merge branch 'master' into issue-label
2 parents c1cf10b + a29363b commit 7c3ba8b

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

src/scripts/scrumHelper.js

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -937,7 +937,7 @@ ${userReason}`;
937937
let prText = '';
938938
prText +=
939939
"<a href='" + pr_arr.html_url + "' target='_blank'>#" + pr_arr.number + '</a> (' + pr_arr.title + ') ';
940-
if (pr_arr.state === 'open') prText += issue_opened_button;
940+
if (showOpenLabel && pr_arr.state === 'open') prText += issue_opened_button;
941941
// Do not show closed label for reviewed PRs
942942
prText += '&nbsp;&nbsp;';
943943
repoLi += prText;
@@ -955,7 +955,7 @@ ${userReason}`;
955955
'</a> (' +
956956
pr_arr1.title +
957957
') ';
958-
if (pr_arr1.state === 'open') prText1 += issue_opened_button;
958+
if (showOpenLabel && pr_arr1.state === 'open') prText1 += issue_opened_button;
959959
// Do not show closed label for reviewed PRs
960960
prText1 += '&nbsp;&nbsp;</li>';
961961
repoLi += prText1;
@@ -1087,9 +1087,9 @@ ${userReason}`;
10871087
} else { }
10881088
const prAction = isNewPR ? 'Made PR' : 'Existing PR';
10891089
if (isDraft) {
1090-
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a> ${pr_draft_button}</li>`;
1090+
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a>${showOpenLabel ? ' ' + pr_draft_button : ''}</li>`;
10911091
} else if (item.state === 'open') {
1092-
li = `<li><i>(${project})</i> - ${prAction} (#${number}) - <a href='${html_url}'>${title}</a> ${pr_open_button}`;
1092+
li = `<li><i>(${project})</i> - ${prAction} (#${number}) - <a href='${html_url}'>${title}</a>${showOpenLabel ? ' ' + pr_open_button : ''}`;
10931093
if (showCommits && item._allCommits && item._allCommits.length && !isNewPR) {
10941094
log(`[PR DEBUG] Rendering commits for existing PR #${number}:`, item._allCommits);
10951095
item._allCommits.forEach(commit => {
@@ -1106,9 +1106,9 @@ ${userReason}`;
11061106
merged = mergedStatusResults[`${owner}/${repo}#${number}`];
11071107
}
11081108
if (merged === true) {
1109-
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a> ${pr_merged_button}</li>`;
1109+
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a>${showOpenLabel ? ' ' + pr_merged_button : ''}</li>`;
11101110
} else {
1111-
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a> ${pr_closed_button}</li>`;
1111+
li = `<li><i>(${project})</i> - Made PR (#${number}) - <a href='${html_url}'>${title}</a>${showOpenLabel ? ' ' + pr_closed_button : ''}</li>`;
11121112
}
11131113
}
11141114
lastWeekArray.push(li);
@@ -1125,15 +1125,15 @@ ${userReason}`;
11251125
html_url +
11261126
"' target='_blank'>" +
11271127
title +
1128-
'</a> ' +
1129-
issue_opened_button +
1128+
'</a>' + (showOpenLabel ? ' ' + issue_opened_button : '') +
11301129
'&nbsp;&nbsp;</li>';
11311130
nextWeekArray.push(li2);
11321131
}
11331132
if (item.state === 'open') {
1134-
li = `<li><i>(${project})</i> - Opened Issue(#${number}) - <a href='${html_url}'>${title}</a> ${issue_opened_button}</li>`;
1133+
li = `<li><i>(${project})</i> - Opened Issue(#${number}) - <a href='${html_url}'>${title}</a>${showOpenLabel ? ' ' + issue_opened_button : ''}</li>`;
11351134
} else if (item.state === 'closed') {
11361135

1136+
11371137
// Use state_reason to distinguish closure reason
11381138
if (item.state_reason === 'completed') {
11391139
li = `<li><i>(${project})</i> - Opened Issue(#${number}) - <a href='${html_url}'>${title}</a> ${issue_closed_completed_button}</li>`;
@@ -1143,6 +1143,7 @@ ${userReason}`;
11431143
li = `<li><i>(${project})</i> - Opened Issue(#${number}) - <a href='${html_url}'>${title}</a> ${issue_closed_button}</li>`;
11441144
}
11451145

1146+
11461147
} else {
11471148
li =
11481149
'<li><i>(' +

0 commit comments

Comments
 (0)