Merge pull request #1371 from topcoder-platform/pm-3141_3

fix(PM-3141): edit mode for copilot users

Author: hentrymartin

.circleci/config.yml

@@ -226,6 +226,7 @@ workflows:
                       branches:
                           only:
                               - dev
+                              - pm-3141_3
 
             - deployQa:
                   context: org-global

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

@@ -27,7 +27,6 @@ import styles from './ReviewViewer.module.scss'
 const ReviewViewer: FC = () => {
     const navigate = useAppNavigate()
     const [searchParams] = useSearchParams()
-    const [respondToAppeals, setRespondToAppeals] = useState(searchParams.get('respondToAppeals') === 'true')
     const {
         reviewId,
         setReviewStatus,
@@ -43,7 +42,8 @@ const ReviewViewer: FC = () => {
     }: useRoleProps = useRole()
     const [showCloseConfirmation, setShowCloseConfirmation] = useState<boolean>(false)
     const [isChanged, setIsChanged] = useState(false)
-    const [isManagerEdit, setIsManagerEdit] = useState(false)
+    const respondToAppeals = searchParams.get('respondToAppeals') === 'true'
+    const [isManagerEdit, setIsManagerEdit] = useState(respondToAppeals)
 
     const {
         challengeInfo,
@@ -206,11 +206,8 @@ const ReviewViewer: FC = () => {
     ])
 
     useEffect(() => {
-        if (!canEditScorecard && isManagerEdit) {
+        if (!canEditScorecard && isManagerEdit && !respondToAppeals) {
             setIsManagerEdit(false)
-        } else if (!isManagerEdit && respondToAppeals) {
-            setIsManagerEdit(true)
-            setRespondToAppeals(false)
         }
     }, [canEditScorecard, isManagerEdit, respondToAppeals])