Skip to content

Commit 3d4cb57

Browse files
authored
Merge pull request #2892 from objectcomputing/feature-2888/view-closed-self-reviews
Feature 2888/view closed self reviews
2 parents 4fc9065 + 317298c commit 3d4cb57

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

web-ui/src/components/reviews/TeamReviews.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ const TeamReviews = ({ onBack, periodId }) => {
172172
useEffect(() => {
173173
const myId = currentUser?.id;
174174
const supervisors = selectSupervisors(state);
175-
const isManager = supervisors.some(s => s.id === myId);
175+
const isManager = supervisors.some(s => s?.id === myId);
176176
const period = selectReviewPeriod(state, periodId);
177177
if (period) {
178178
setApprovalState(period.reviewStatus === ReviewStatus.AWAITING_APPROVAL);

web-ui/src/components/reviews/periods/ReviewPeriods.jsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,8 +327,13 @@ const ReviewPeriods = ({ onPeriodSelected, mode }) => {
327327
case ReviewStatus.OPEN:
328328
onPeriodSelected(id);
329329
break;
330+
case ReviewStatus.CLOSED:
331+
if (mode === selfReviewMode) {
332+
onPeriodSelected(id);
333+
}
334+
break;
330335
default:
331-
// We do nothing if the status is CLOSED or UNKNOWN.
336+
// We do nothing if the status is UNKNOWN.
332337
break;
333338
}
334339
},

0 commit comments

Comments
 (0)