Merge pull request #1222 from topcoder-platform/dev

[PROD RELEASE] - Copilot Portal tweaks & tech dept clean up

Author: kkartunov

src/apps/accounts/src/lib/assets/security/dicelogo.png

@@ -1,17 +1,9 @@
-import { ReactComponent as MFAImage } from './mfa.svg'
 import { ReactComponent as AppleStore } from './apple-store.svg'
 import credentialImage from './credential.png'
-import diceIdLogo from './dicelogo.png'
-import diceIdLogoBig from './dicelogobig.png'
-import diceIdLogoSmall from './dicelogosmall.png'
 import googlePlay from './google-play.png'
 
 export {
     AppleStore,
     credentialImage,
-    diceIdLogo,
-    diceIdLogoBig,
-    diceIdLogoSmall,
     googlePlay,
-    MFAImage,
 }

src/apps/accounts/src/lib/assets/security/dicelogobig.png

@@ -86,12 +86,8 @@ const CopilotOpportunityDetails: FC<{}> = () => {
     const [activeTab, setActiveTab]: [string, Dispatch<SetStateAction<string>>] = useState<string>(activeTabHash)
 
     useEffect(() => {
-        if (isAdminOrPM) {
-            setActiveTab(activeTabHash)
-        } else {
-            setActiveTab('0')
-        }
-    }, [activeTabHash, isAdminOrPM])
+        setActiveTab(activeTabHash)
+    }, [activeTabHash])
 
     const handleTabChange = useCallback((tabId: string): void => {
         setActiveTab(tabId)
@@ -286,7 +282,7 @@ const CopilotOpportunityDetails: FC<{}> = () => {
                         <TabsNavbar
                             defaultActive={activeTab}
                             onChange={handleTabChange}
-                            tabs={getCopilotDetailsTabsConfig(isAdminOrPM, copilotApplications?.length || 0)}
+                            tabs={getCopilotDetailsTabsConfig(copilotApplications?.length || 0)}
                         />
                     )
                 }
@@ -297,7 +293,7 @@ const CopilotOpportunityDetails: FC<{}> = () => {
                         opportunity={opportunity}
                         members={members}
                         onApplied={onApplied}
-                        isAdminOrPM
+                        isAdminOrPM={isAdminOrPM}
                     />
                 )}
 

src/apps/accounts/src/lib/assets/security/dicelogosmall.png

@@ -5,7 +5,7 @@ export enum CopilotDetailsTabViews {
     applications = '1',
 }
 
-export const getCopilotDetailsTabsConfig = (isAdminOrPM: boolean, count: number): TabsNavItem[] => (isAdminOrPM ? [
+export const getCopilotDetailsTabsConfig = (count: number): TabsNavItem[] => ([
     {
         id: CopilotDetailsTabViews.details,
         title: 'Details',
@@ -18,11 +18,6 @@ export const getCopilotDetailsTabsConfig = (isAdminOrPM: boolean, count: number)
         id: CopilotDetailsTabViews.applications,
         title: 'Applications',
     },
-] : [
-    {
-        id: CopilotDetailsTabViews.details,
-        title: 'Details',
-    },
 ])
 
 export const CopilotDetailsTabsConfig: TabsNavItem[] = [

src/apps/accounts/src/lib/assets/security/index.ts

@@ -1,12 +1,10 @@
-import { Dispatch, FC, SetStateAction, useCallback, useContext, useState } from 'react'
+import { FC, useCallback, useContext } from 'react'
 import { NavigateFunction, useNavigate } from 'react-router-dom'
 
 import { Button } from '~/libs/ui'
 import { profileContext, ProfileContextData } from '~/libs/core'
 
 import { getAuthenticateAndEnrollRoute, getTCACertificationEnrollPath } from '../../learn.routes'
-import { LearnConfig } from '../../config'
-import { DiceModal } from '../../course-details/course-curriculum/dice-modal'
 
 interface EnrollCtaBtnProps {
     certification: string
@@ -15,15 +13,9 @@ interface EnrollCtaBtnProps {
 const EnrollCtaBtn: FC<EnrollCtaBtnProps> = (props: EnrollCtaBtnProps) => {
     const navigate: NavigateFunction = useNavigate()
     const { initialized: profileReady, profile }: ProfileContextData = useContext(profileContext)
-    const [isDiceModalOpen, setIsDiceModalOpen]: [boolean, Dispatch<SetStateAction<boolean>>]
-        = useState<boolean>(false)
 
     const isLoggedIn: boolean = profileReady && !!profile
 
-    function onDiceModalClose(): void {
-        setIsDiceModalOpen(false)
-    }
-
     /**
      * Handle user click on start course/resume/login button
      */
@@ -37,15 +29,8 @@ const EnrollCtaBtn: FC<EnrollCtaBtnProps> = (props: EnrollCtaBtnProps) => {
             return
         }
 
-        // if the user is wipro and s/he hasn't set up DICE,
-        // let the user know
-        if (LearnConfig.REQUIRE_DICE_ID && profile?.isWipro && !profile.diceEnabled) {
-            setIsDiceModalOpen(true)
-            return
-        }
-
         navigate(getTCACertificationEnrollPath(props.certification))
-    }, [isLoggedIn, profile?.isWipro, profile?.diceEnabled, props, navigate])
+    }, [isLoggedIn, props, navigate])
 
     return (
         <>
@@ -55,11 +40,6 @@ const EnrollCtaBtn: FC<EnrollCtaBtnProps> = (props: EnrollCtaBtnProps) => {
                 label={isLoggedIn ? 'Enroll Now' : 'Log in to enroll'}
                 onClick={handleEnrollClick}
             />
-
-            <DiceModal
-                isOpen={isDiceModalOpen}
-                onClose={onDiceModalClose}
-            />
         </>
     )
 }

src/apps/accounts/src/lib/assets/security/mfa.svg

@@ -7,5 +7,4 @@ export interface LearnConfigModel {
         value: string,
     }
     CLIENT: string
-    REQUIRE_DICE_ID: boolean | undefined
 }

src/apps/copilots/src/pages/copilot-opportunity-details/index.tsx

@@ -1,5 +1,4 @@
 import { EnvironmentConfig } from '~/config'
-import { getReactEnv } from '~/config/environments/react-env'
 
 import { LearnConfigModel } from './learn-config.model'
 
@@ -14,5 +13,4 @@ export const LearnConfigDefault: LearnConfigModel = {
         value: 'certificate-container',
     },
     CLIENT: 'https://fcc.topcoder-dev.com:4431',
-    REQUIRE_DICE_ID: `${getReactEnv<string>('REQUIRE_DICE_ID', '')}` === 'true',
 }