Skip to content

Commit 606cc39

Browse files
committed
PM-684 - make job description a required field for Taas Projects
1 parent 01b0ee1 commit 606cc39

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

src/containers/TaaSProjectForm/index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -401,6 +401,7 @@ const TaaSProjectForm = ({
401401
errorMsg={getFormError(
402402
`jobs[${index}].description`
403403
)}
404+
isRequired
404405
>
405406
<FieldDescription
406407
value={job.description}

src/util/validation.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,29 +28,29 @@ export const taaSProjectFormValidationSchema = Yup.object({
2828
.required('Please enter project title'),
2929
jobs: Yup.array().of(
3030
Yup.object().shape({
31-
title: Yup.string().required('Please, enter job title'),
31+
title: Yup.string().required('Please enter job title'),
3232
people: Yup.number()
33-
.min(1, 'Please, choose at least one people')
34-
.test('number', 'Please, choose at least one people', isValidNumber),
35-
role: Yup.object().test('role', 'Please, choose role', isValidSelect),
33+
.min(1, 'Please choose at least one people')
34+
.test('number', 'Please choose at least one people', isValidNumber),
35+
role: Yup.object().test('role', 'Please choose role', isValidSelect),
3636
duration: Yup.number()
37-
.min(4, 'Please, choose at least 4 weeks')
38-
.test('number', 'Please, choose at least 4 weeks', isValidNumber),
37+
.min(4, 'Please choose at least 4 weeks')
38+
.test('number', 'Please choose at least 4 weeks', isValidNumber),
3939
workload: Yup.object().test(
4040
'workload',
41-
'Please, choose workload',
41+
'Please choose workload',
4242
isValidSelect
4343
),
44-
description: Yup.string().optional(),
44+
description: Yup.string().required('Please enter a job description'),
4545
skills: Yup.array()
4646
.of(
4747
Yup.object().shape({
4848
name: Yup.string(),
4949
skillId: Yup.string()
5050
})
5151
)
52-
.required('Required', 'Please, choose at least one skill.')
53-
.min(1, 'Please, choose at least one skill.')
52+
.required('Required', 'Please choose at least one skill.')
53+
.min(1, 'Please choose at least one skill.')
5454
})
5555
)
5656
})

0 commit comments

Comments
 (0)