From 7ffccec1029bb5536931d3e4e2a6191f100550fb Mon Sep 17 00:00:00 2001
From: dhruvjoshi <dhruv.joshi@navi.com>
Date: Tue, 5 Mar 2024 14:39:30 +0530
Subject: [PATCH] INFRA-2874 | pr comment resolve

---
 src/Pages/TeamRevamp/partials/SlackDetails/index.tsx | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

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<HTMLInputElement>,
   ): void => {
@@ -114,16 +117,14 @@ const TeamDetails: React.FC = () => {
   }) as MutableRefObject<HTMLDivElement>;
 
   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 (
     <div className={styles['team-details-wrapper']}>
       <Typography variant="h1" className={styles['team-name']}>