Merge pull request #1361 from topcoder-platform/pm-3141

fix(PM-3141): allow only admins and reviewer to respond to appeals

Author: hentrymartin

src/apps/review/src/lib/components/Scorecard/ScorecardViewer/ScorecardQuestion/ReviewResponse/ReviewComment/ReviewComment.tsx

@@ -10,6 +10,7 @@ import { get, includes } from 'lodash'
 import { yupResolver } from '@hookform/resolvers/yup'
 import { IconAppeal, IconEdit } from '~/apps/review/src/lib/assets/icons'
 import { ADMIN, COPILOT, REVIEWER } from '~/apps/review/src/config/index.config'
+import { useRolePermissions, UseRolePermissionsResult } from '~/apps/review/src/lib/hooks'
 
 import {
     AppealInfo,
@@ -44,6 +45,7 @@ const ReviewComment: FC<ReviewCommentProps> = props => {
         addAppeal,
         isSavingAppeal,
     }: ScorecardViewerContextValue = useScorecardViewerContext()
+    const { isAdmin, hasReviewerRole }: UseRolePermissionsResult = useRolePermissions()
 
     const { challengeInfo }: ChallengeDetailContextModel = useContext(
         ChallengeDetailContext,
@@ -175,7 +177,7 @@ const ReviewComment: FC<ReviewCommentProps> = props => {
                         appeal={props.appeal}
                         reviewItem={props.reviewItem}
                         scorecardQuestion={props.question}
-                        canRespondToAppeal={isReviewerRole}
+                        canRespondToAppeal={isAdmin || hasReviewerRole}
                     >
                         {isSubmitter && canAddAppeal && (
                             <div className={styles.blockBtns}>

src/apps/review/src/lib/components/TableAppealsResponse/TableAppealsResponse.tsx

@@ -107,7 +107,7 @@ export const TableAppealsResponse: FC<TableAppealsResponseProps> = (props: Table
     const canViewAsReviewer = hasReviewerRole
     const canViewAsSubmitter = hasSubmitterRole
     const canRender = canViewAllAppeals || canViewAsReviewer || canViewAsSubmitter
-    const canRespondToAppeals = hasReviewerRole
+    const canRespondToAppeals = hasReviewerRole || isAdmin
 
     const isAppealsResponsePhaseOpen = useMemo<boolean>(
         () => (challengeInfo?.phases ?? []).some(phase => phase?.name?.toLowerCase() === 'appeals response'