Handle the show_data_dashboard flag as string or boolean

topcoder-platform · Dec 14, 2023 · a36a464 · a36a464
1 parent 51b997f
commit a36a464
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/src/components/ChallengeEditor/ChallengeView/index.js b/src/components/ChallengeEditor/ChallengeView/index.js
@@ -102,7 +102,9 @@ const ChallengeView = ({
   const showCheckpointPrizes = _.get(challenge, 'timelineTemplateId') === MULTI_ROUND_CHALLENGE_TEMPLATE_ID
   const isDataScience = challenge.trackId === DS_TRACK_ID
   const useDashboardData = _.find(challenge.metadata, { name: 'show_data_dashboard' })
-  const useDashboard = useDashboardData ? (useDashboardData.value === 'true') : false
+  const useDashboard = useDashboardData ? 
+    (_.isString(useDashboardData.value) && useDashboardData.value === "true") || 
+    (_.isBoolean(useDashboardData.value) && useDashboardData.value) : false
 
   return (
     <div className={styles.wrapper}>

diff --git a/src/components/ChallengeEditor/index.js b/src/components/ChallengeEditor/index.js
@@ -1069,8 +1069,11 @@ class ChallengeEditor extends Component {
       }
       let useDashboard = _.find(challengeMetadata, { name: 'show_data_dashboard' })
       if (useDashboard === undefined) {
-        useDashboard = { name: 'show_data_dashboard', value: true }
-      }
+        useDashboard = { name: 'show_data_dashboard', value: "false" }
+      } else if(_.isBoolean(useDashboard.value)){
+        useDashboard = { name: 'show_data_dashboard', value: _.toString(useDashboard.value) }
+      } 
+
       newChallenge.metadata.push(useDashboard)
     }
     try {
@@ -1646,7 +1649,11 @@ class ChallengeEditor extends Component {
     const showCheckpointPrizes = challenge.timelineTemplateId === MULTI_ROUND_CHALLENGE_TEMPLATE_ID
     const showDashBoard = (challenge.trackId === DS_TRACK_ID && isChallengeType) || (isDevChallenge && isMM)
     const useDashboardData = _.find(challenge.metadata, { name: 'show_data_dashboard' })
-    const useDashboard = useDashboardData ? useDashboardData.value : true
+
+
+    const useDashboard = useDashboardData ? 
+          (_.isString(useDashboardData.value) && useDashboardData.value === "true") || 
+          (_.isBoolean(useDashboardData.value) && useDashboardData.value) : false
     const workTypes = getDomainTypes(challenge.trackId)
     const filteredTypes = metadata.challengeTypes.filter(type => workTypes.includes(type.abbreviation))