diff --git a/src/Pages/TeamRevamp/partials/SlackDetails/index.tsx b/src/Pages/TeamRevamp/partials/SlackDetails/index.tsx index 54046b3..1ea42fe 100644 --- a/src/Pages/TeamRevamp/partials/SlackDetails/index.tsx +++ b/src/Pages/TeamRevamp/partials/SlackDetails/index.tsx @@ -51,6 +51,9 @@ const TeamDetails: React.FC = () => { }); }, [teamData]); + const isNeitherParticipantNorAdmin: boolean = + !isUserParticipant(teamMembers, userEmail) && !Role.includes('Admin'); + const handleslackChannelId = ( e: React.ChangeEvent, ): void => { @@ -114,16 +117,14 @@ const TeamDetails: React.FC = () => { }) as MutableRefObject; const handleOncallPicker = (): void => { - if (!isUserParticipant(teamMembers, userEmail) && !Role.includes('Admin')) - return; + if (isNeitherParticipantNorAdmin) return; dispatchData({ type: actionTypes.SET_OPEN_ONCALL, payload: !state.openOnCall, }); }; const handleOpenPsecOnCallPicker = (): void => { - if (!isUserParticipant(teamMembers, userEmail) && !Role.includes('Admin')) - return; + if (isNeitherParticipantNorAdmin) return; dispatchData({ type: actionTypes.SET_PSEC_OPEN_ONCALL, payload: !state.openPsecOnCall, @@ -151,9 +152,6 @@ const TeamDetails: React.FC = () => { ? styles['hint-text'] : styles['hint-text-disabled']; - const isNeitherParticipantNorAdmin: boolean = - !isUserParticipant(teamMembers, userEmail) && !Role.includes('Admin'); - return (