Skip to content

Commit 9be0d53

Browse files
committed
Added a check for checkins creation permission.
1 parent 70c7383 commit 9be0d53

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

web-ui/src/context/selectors.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,10 @@ export const selectCanUpdateCheckinsPermission = hasPermission(
206206
'CAN_UPDATE_CHECKINS'
207207
);
208208

209+
export const selectCanCreateCheckinsPermission = hasPermission(
210+
'CAN_CREATE_CHECKINS'
211+
);
212+
209213
export const selectCanUpdateAllCheckinsPermission = hasPermission(
210214
'CAN_UPDATE_ALL_CHECKINS'
211215
);

web-ui/src/pages/CheckinsPage.jsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import {
1313
selectProfile,
1414
selectCheckinsForMember,
1515
selectCanViewCheckinsPermission,
16+
selectCanCreateCheckinsPermission,
1617
selectCanUpdateCheckinsPermission,
1718
selectCanViewPrivateNotesPermission,
1819
selectCanUpdateAllCheckinsPermission,
@@ -159,7 +160,8 @@ const CheckinsPage = () => {
159160
aria-describedby="checkin-tooltip-wrapper"
160161
className="create-checkin-tooltip-wrapper"
161162
>
162-
{(updateAll || isPdl || currentUserId === memberId) && (
163+
{(updateAll || isPdl || currentUserId === memberId) &&
164+
selectCanCreateCheckinsPermission(state) && (
163165
<Button
164166
disabled={hasOpenCheckins}
165167
className={classes.addButton}

0 commit comments

Comments
 (0)