Merge pull request #1518 from topcoder-platform/PM-4075

PM-4075 Fix manager comment access

Author: kkartunov

src/apps/review/src/pages/reviews/components/ReviewViewer/ReviewViewer.tsx

@@ -43,7 +43,39 @@ const ReviewViewer: FC = () => {
     const [showCloseConfirmation, setShowCloseConfirmation] = useState<boolean>(false)
     const [isChanged, setIsChanged] = useState(false)
     const respondToAppeals = searchParams.get('respondToAppeals') === 'true'
-    const [isManagerEdit, setIsManagerEdit] = useState(respondToAppeals)
+    const hasChallengeAdminRole = useMemo(
+        () => myChallengeResources.some(
+            resource => resource.roleName?.toLowerCase() === ADMIN.toLowerCase(),
+        ),
+        [myChallengeResources],
+    )
+
+    const hasTopcoderAdminRole = useMemo(
+        () => myChallengeRoles.some(
+            role => role?.toLowerCase()
+                .includes('admin'),
+        ),
+        [myChallengeRoles],
+    )
+
+    const hasChallengeManagerRole = useMemo(
+        () => myChallengeResources.some(
+            resource => resource.roleName?.toLowerCase() === MANAGER.toLowerCase(),
+        ),
+        [myChallengeResources],
+    )
+
+    const canManagerEdit = useMemo(
+        () => hasChallengeAdminRole
+        || hasTopcoderAdminRole
+        || hasChallengeManagerRole,
+        [
+            hasChallengeAdminRole,
+            hasTopcoderAdminRole,
+            hasChallengeManagerRole,
+        ],
+    )
+    const [isManagerEdit, setIsManagerEdit] = useState(respondToAppeals && canManagerEdit)
 
     const {
         challengeInfo,
@@ -148,28 +180,6 @@ const ReviewViewer: FC = () => {
         })
     }, [challengeInfo?.id, mutate, navigate])
 
-    const hasChallengeAdminRole = useMemo(
-        () => myChallengeResources.some(
-            resource => resource.roleName?.toLowerCase() === ADMIN.toLowerCase(),
-        ),
-        [myChallengeResources],
-    )
-
-    const hasTopcoderAdminRole = useMemo(
-        () => myChallengeRoles.some(
-            role => role?.toLowerCase()
-                .includes('admin'),
-        ),
-        [myChallengeRoles],
-    )
-
-    const hasChallengeManagerRole = useMemo(
-        () => myChallengeResources.some(
-            resource => resource.roleName?.toLowerCase() === MANAGER.toLowerCase(),
-        ),
-        [myChallengeResources],
-    )
-
     const hasChallengeCopilotRole = useMemo(
         () => myChallengeResources.some(
             resource => resource.roleName?.toLowerCase() === COPILOT.toLowerCase(),