Skip to content

Commit 505201a

Browse files
committed
Changed type value and removed Review typeMap labels
1 parent 431c299 commit 505201a

File tree

2 files changed

+12
-13
lines changed

2 files changed

+12
-13
lines changed

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

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import PropTypes from 'prop-types'
33
import { connect } from 'react-redux'
44
import cn from 'classnames'
55
import { PrimaryButton, OutlineButton } from '../../Buttons'
6-
import { REVIEW_OPPORTUNITY_TYPES, VALIDATION_VALUE_TYPE } from '../../../config/constants'
6+
import { REVIEW_OPPORTUNITY_TYPE_LABELS, REVIEW_OPPORTUNITY_TYPES, VALIDATION_VALUE_TYPE } from '../../../config/constants'
77
import { loadScorecards, loadDefaultReviewers, loadWorkflows } from '../../../actions/challenges'
88
import styles from './ChallengeReviewer-Field.module.scss'
99
import { convertDollarToInteger, validateValue } from '../../../util/input-check'
@@ -126,7 +126,7 @@ class ChallengeReviewerField extends Component {
126126
phaseId: (defaultReviewer && defaultReviewer.phaseId) || (firstReviewPhase ? (firstReviewPhase.id || firstReviewPhase.phaseId) : ''),
127127
basePayment: (defaultReviewer && defaultReviewer.basePayment) || '0',
128128
incrementalPayment: (defaultReviewer && defaultReviewer.incrementalPayment) || 0,
129-
type: isAIReviewer ? null : ((defaultReviewer && defaultReviewer.opportunityType) || REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW)
129+
type: isAIReviewer ? undefined : ((defaultReviewer && defaultReviewer.opportunityType) || REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW)
130130
}
131131

132132
if (isAIReviewer) {
@@ -285,7 +285,7 @@ class ChallengeReviewerField extends Component {
285285
phaseId: currentReviewer.phaseId,
286286
basePayment: currentReviewer.basePayment || '0',
287287
incrementalPayment: currentReviewer.incrementalPayment || 0,
288-
type: isAI ? null : (currentReviewer.type || REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW)
288+
type: isAI ? undefined : (currentReviewer.type || REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW)
289289
}
290290

291291
if (isAI) {
@@ -459,16 +459,7 @@ class ChallengeReviewerField extends Component {
459459
<label>Review Type:</label>
460460
{readOnly ? (
461461
<span>
462-
{(() => {
463-
const typeMap = {
464-
[REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW]: 'Regular Review',
465-
[REVIEW_OPPORTUNITY_TYPES.COMPONENT_DEV_REVIEW]: 'Component Dev Review',
466-
[REVIEW_OPPORTUNITY_TYPES.SPEC_REVIEW]: 'Spec Review',
467-
[REVIEW_OPPORTUNITY_TYPES.ITERATIVE_REVIEW]: 'Iterative Review',
468-
[REVIEW_OPPORTUNITY_TYPES.SCENARIOS_REVIEW]: 'Scenarios Review'
469-
}
470-
return typeMap[reviewer.type] || 'Regular Review'
471-
})()}
462+
{ REVIEW_OPPORTUNITY_TYPE_LABELS[reviewer.type] || 'Regular Review'}
472463
</span>
473464
) : (
474465
<select

src/config/constants.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,14 @@ export const REVIEW_OPPORTUNITY_TYPES = {
247247
SCENARIOS_REVIEW: 'SCENARIOS_REVIEW'
248248
}
249249

250+
export const REVIEW_OPPORTUNITY_TYPE_LABELS = {
251+
[REVIEW_OPPORTUNITY_TYPES.REGULAR_REVIEW]: 'Regular Review',
252+
[REVIEW_OPPORTUNITY_TYPES.COMPONENT_DEV_REVIEW]: 'Component Dev Review',
253+
[REVIEW_OPPORTUNITY_TYPES.SPEC_REVIEW]: 'Spec Review',
254+
[REVIEW_OPPORTUNITY_TYPES.ITERATIVE_REVIEW]: 'Iterative Review',
255+
[REVIEW_OPPORTUNITY_TYPES.SCENARIOS_REVIEW]: 'Scenarios Review'
256+
}
257+
250258
export const ROUND_TYPES = {
251259
'SINGLE_ROUND': 'Single round',
252260
'TWO_ROUNDS': 'Two rounds'

0 commit comments

Comments
 (0)