Skip to content

Commit 8c8fc41

Browse files
authored
Merge pull request #361 from topcoder-platform/change-validatations-in-job-jc
Change validatations in job jc
2 parents 99f4574 + 01a2e08 commit 8c8fc41

File tree

2 files changed

+19
-15
lines changed

2 files changed

+19
-15
lines changed

src/services/JobCandidateService.js

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -195,18 +195,22 @@ async function fullyUpdateJobCandidate (currentUser, id, data) {
195195
return updateJobCandidate(currentUser, id, data)
196196
}
197197

198-
fullyUpdateJobCandidate.schema = Joi.object().keys({
199-
currentUser: Joi.object().required(),
200-
id: Joi.string().uuid().required(),
201-
data: Joi.object().keys({
202-
jobId: Joi.string().uuid().required(),
203-
userId: Joi.string().uuid().required(),
204-
status: Joi.jobCandidateStatus().default('open'),
205-
externalId: Joi.string().allow(null).default(null),
206-
resume: Joi.string().uri().allow('').allow(null).default(null),
207-
remark: Joi.string().allow('').allow(null).default(null)
208-
}).required()
209-
}).required()
198+
fullyUpdateJobCandidate.schema = Joi.object()
199+
.keys({
200+
currentUser: Joi.object().required(),
201+
id: Joi.string().uuid().required(),
202+
data: Joi.object()
203+
.keys({
204+
jobId: Joi.string().uuid().required(),
205+
userId: Joi.string().uuid().required(),
206+
status: Joi.jobCandidateStatus().default("open"),
207+
externalId: Joi.string().allow(null).default(null),
208+
resume: Joi.string().uri().allow("").allow(null).default(null),
209+
remark: Joi.stringAllowEmpty().allow(null),
210+
})
211+
.required(),
212+
})
213+
.required();
210214

211215
/**
212216
* Delete jobCandidate by id

src/services/JobService.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,9 +208,9 @@ createJob.schema = Joi.object()
208208
minSalary: Joi.number().integer().allow(null),
209209
maxSalary: Joi.number().integer().allow(null),
210210
hoursPerWeek: Joi.number().integer().allow(null),
211-
jobLocation: Joi.string().allow(null).allow(''),
212-
jobTimezone: Joi.string().allow(null).allow(''),
213-
currency: Joi.string().allow(null).allow(''),
211+
jobLocation: Joi.stringAllowEmpty().allow(null),
212+
jobTimezone: Joi.stringAllowEmpty().allow(null),
213+
currency: Joi.stringAllowEmpty().allow(null),
214214
roleIds: Joi.array().items(Joi.string().uuid().required())
215215
})
216216
.required()

0 commit comments

Comments
 (0)