Merge pull request #1665 from topcoder-platform/develop

Hotifx - disable alphanumeric characters in challenge title

Author: jmgasper

.gitignore

@@ -25,7 +25,7 @@ yarn-debug.log*
 yarn-error.log*
 
 *.env
-
+*.pem
 *.vscode
 
 # e2e test case

config/constants/development.js

@@ -19,11 +19,9 @@ module.exports = {
   CHALLENGE_TRACKS_URL: `${DEV_API_HOSTNAME}/v5/challenge-tracks`,
   CHALLENGE_PHASES_URL: `${DEV_API_HOSTNAME}/v5/challenge-phases`,
   CHALLENGE_TIMELINES_URL: `${DEV_API_HOSTNAME}/v5/challenge-timelines`,
-  COPILOTS_URL: `https://copilots.${DOMAIN}`,
   PROJECT_API_URL: `${DEV_API_HOSTNAME}/v5/projects`,
   GROUPS_API_URL: `${DEV_API_HOSTNAME}/v5/groups`,
   TERMS_API_URL: `${DEV_API_HOSTNAME}/v5/terms`,
-  MEMBERS_API_URL: `${DEV_API_HOSTNAME}/v5/members`,
   RESOURCES_API_URL: `${DEV_API_HOSTNAME}/v5/resources`,
   RESOURCE_ROLES_API_URL: `${DEV_API_HOSTNAME}/v5/resource-roles`,
   SUBMISSIONS_API_URL: `${DEV_API_HOSTNAME}/v5/submissions`,

package-lock.json

@@ -4,22 +4,47 @@ import styles from './ChallengeName-Field.module.scss'
 import cn from 'classnames'
 
 const ChallengeNameField = ({ challenge, onUpdateInput }) => {
+  const handleChange = (e) => {
+    // Remove any characters that are NOT letters, numbers, or spaces
+    const sanitizedValue = e.target.value.replace(/[^a-zA-Z0-9 ]/g, '')
+    onUpdateInput({
+      target: {
+        name: e.target.name,
+        value: sanitizedValue
+      }
+    })
+  }
+
   return (
     <>
       <div className={styles.row}>
         <div className={cn(styles.field, styles.col1)}>
-          <label htmlFor='challengeName'>Work Name <span>*</span> :</label>
+          <label htmlFor='challengeName'>
+            Work Name <span>*</span> :
+          </label>
         </div>
         <div className={cn(styles.field, styles.col2)}>
-          <input className={styles.challengeName} id='name' name='name' type='text' placeholder='Work Name' value={challenge.name} maxLength='200' required onChange={onUpdateInput} />
+          <input
+            className={styles.challengeName}
+            id='name'
+            name='name'
+            type='text'
+            placeholder='Work Name'
+            value={challenge.name}
+            maxLength='200'
+            required
+            onChange={handleChange}
+          />
         </div>
       </div>
-      { challenge.submitTriggered && !challenge.name && <div className={styles.row}>
-        <div className={cn(styles.field, styles.col1)} />
-        <div className={cn(styles.field, styles.col2, styles.error)}>
-          Work Name is required field
+      {challenge.submitTriggered && !challenge.name && (
+        <div className={styles.row}>
+          <div className={cn(styles.field, styles.col1)} />
+          <div className={cn(styles.field, styles.col2, styles.error)}>
+            Work Name is required field
+          </div>
         </div>
-      </div> }
+      )}
     </>
   )
 }

src/components/ChallengeEditor/ChallengeName-Field/index.js

@@ -7,7 +7,7 @@ import PropTypes from 'prop-types'
 import { Helmet } from 'react-helmet'
 import { Link } from 'react-router-dom'
 import ProjectStatus from './ProjectStatus'
-import { PROJECT_ROLES, PROJECT_STATUS, COPILOTS_URL } from '../../config/constants'
+import { PROJECT_ROLES, PROJECT_STATUS, COPILOTS_URL, CHALLENGE_STATUS } from '../../config/constants'
 import { PrimaryButton, OutlineButton } from '../Buttons'
 import ChallengeList from './ChallengeList'
 import styles from './ChallengesComponent.module.scss'
@@ -53,6 +53,12 @@ const ChallengesComponent = ({
   const isReadOnly = checkReadOnlyRoles(auth.token) || loginUserRoleInProject === PROJECT_ROLES.READ
   const isAdminOrCopilot = checkAdminOrCopilot(auth.token, activeProject)
 
+  const projectStatus = activeProject && activeProject.status
+    ? activeProject.status.toUpperCase()
+    : ''
+  const isCompletedOrCancelled =
+  projectStatus === CHALLENGE_STATUS.CANCELLED || projectStatus === CHALLENGE_STATUS.COMPLETED
+
   useEffect(() => {
     const loggedInUser = auth.user
     const projectMembers = activeProject.members
@@ -94,7 +100,7 @@ const ChallengesComponent = ({
                 className={styles.btnOutline}
               />
             )}
-            {(checkAdmin(auth.token) || checkManager(auth.token)) && (
+            {(checkAdmin(auth.token) || checkManager(auth.token)) && !isCompletedOrCancelled && (
               <OutlineButton
                 text='Request Copilot'
                 type={'info'}