Skip to content

Commit

Permalink
3rd prettier changes
Browse files Browse the repository at this point in the history
  • Loading branch information
jessrey619 committed Nov 15, 2024
1 parent c40620b commit 48da661
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions client/src/pages/notifications.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,9 @@ const Notifications: NextPage = (): JSX.Element => {
startDate: parsedData.StartDate,
endDate: parsedData.EndDate,
leaveType:
notificationType === NOTIFICATION_TYPE.LEAVE || NOTIFICATION_TYPE.LEAVE_RESOLVED
? parsedData.LeaveType
notificationType === NOTIFICATION_TYPE.LEAVE ||
notificationType === NOTIFICATION_TYPE.LEAVE_RESOLVED
? parsedData.LeaveType || null

Check failure on line 85 in client/src/pages/notifications.tsx

View workflow job for this annotation

GitHub Actions / Build frontend

Unexpected nullable string value in conditional. Please handle the nullish/empty cases explicitly

Check failure on line 85 in client/src/pages/notifications.tsx

View workflow job for this annotation

GitHub Actions / Build frontend

Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator
: null,
overtimeId: parsedData.OvertimeId,
managerApproveStatus:
Expand Down

0 comments on commit 48da661

Please sign in to comment.