Skip to content

Commit a2c6d2a

Browse files
authored
Don't show Last Reviewer for Optional/Declined PRs (#167) (#232)
1 parent 66c480b commit a2c6d2a

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/azdo-pr-dashboard.user.js

+6-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// ==UserScript==
22

33
// @name More Awesome Azure DevOps (userscript)
4-
// @version 3.7.0
4+
// @version 3.7.1
55
// @author Alejandro Barreto (NI)
66
// @description Makes general improvements to the Azure DevOps experience, particularly around pull requests. Also contains workflow improvements for NI engineers.
77
// @license MIT
@@ -1469,7 +1469,7 @@
14691469
await annotateFileCountOnPullRequestRow(row, pr);
14701470
await annotateBuildStatusOnPullRequestRow(row, pr);
14711471

1472-
if (votes.userVote === 0 && votes.missingVotes === 1) {
1472+
if (votes.userVote === 0 && votes.missingVotes === 1 && votes.userIsRequired && !votes.userHasDeclined) {
14731473
annotatePullRequestTitle(row, 'repos-pr-list-late-review-pill', 'Last Reviewer', 'Everyone is waiting on you!');
14741474
}
14751475

@@ -1538,11 +1538,15 @@
15381538
missingVotes: 0,
15391539
waitingOrRejectedVotes: 0,
15401540
userVote: 0,
1541+
userIsRequired: true,
1542+
userHasDeclined: false,
15411543
};
15421544

15431545
for (const reviewer of pr.reviewers) {
15441546
if (reviewer.uniqueName === currentUser.uniqueName) {
15451547
votes.userVote = reviewer.vote;
1548+
votes.userIsRequired = reviewer.isRequired;
1549+
votes.userHasDeclined = reviewer.hasDeclined;
15461550
}
15471551
if (reviewer.vote === 0) {
15481552
votes.missingVotes += 1;

0 commit comments

Comments
 (0)