fix unapprove redirect url

Author: stevenfrog

src/apps/admin/src/lib/components/common/Tab/SystemAdminTabs.tsx

@@ -1,4 +1,4 @@
-import { Dispatch, FC, SetStateAction, useMemo, useState } from 'react'
+import { Dispatch, FC, SetStateAction, useEffect, useMemo, useState } from 'react'
 import { NavigateFunction, useLocation, useNavigate } from 'react-router-dom'
 
 import { TabsNavbar } from '~/libs/ui'
@@ -24,6 +24,14 @@ const SystemAdminTabs: FC = () => {
         navigate(childTabId)
     }
 
+    // If url is changed by navigator on different tabs, we need set activeTab
+    useEffect(() => {
+        const pathTabId = getTabIdFromPathName(pathname)
+        if (pathTabId !== activeTab) {
+            setActiveTab(pathTabId)
+        }
+    }, [pathname]) // eslint-disable-line react-hooks/exhaustive-deps
+
     return (
         <div className={styles.container}>
             <TabsNavbar

src/apps/admin/src/review-management/ManageReviewerPage/ManageReviewerPage.tsx

@@ -32,6 +32,7 @@ import {
 } from '../../lib/models'
 import {
     approveApplication,
+    getChallengeByLegacyId,
     getChallengeReviewers,
     getChallengeReviewOpportunities,
     rejectPending,
@@ -76,6 +77,7 @@ export const ManageReviewerPage: FC = () => {
         Array<Reviewer>,
         Dispatch<SetStateAction<Array<Reviewer>>>
     ] = useState<Array<Reviewer>>([])
+    const [challengeUuid, setChallengeUuid] = useState('')
 
     const {
         search: doSearch,
@@ -136,17 +138,25 @@ export const ManageReviewerPage: FC = () => {
     })
 
     const unapprove = useEventCallback((): void => {
-        // how to get challenge Id?
-        // Now we use one specific challenge id for testing
-        const realChallengeId = 'c713e250-ecb4-4192-8717-d607ddda8db4'
-        navigate(`${rootRoute}/challenge-management/${realChallengeId}/manage-user`)
+        if (challengeUuid) {
+            navigate(`${rootRoute}/challenge-management/${challengeUuid}/manage-user`)
+        }
     })
 
     // Init
     useEffect(() => {
         search()
     }, [challengeId]) // eslint-disable-line react-hooks/exhaustive-deps -- missing dependency: search
 
+    // Gets the challenge details by legacyId
+    useEffect(() => {
+        getChallengeByLegacyId(+challengeId)
+            .then(challenge => {
+                setChallengeUuid(challenge.id)
+            })
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- missing dependency: setChallengeUuid
+    }, [challengeId, getChallengeByLegacyId])
+
     // Page change
     const [pageChangeEvent, setPageChangeEvent] = useState(false)
     const previousPageChangeEvent = useRef(false)