Skip to content

Commit a5036b1

Browse files
committed
Fix how we determine if a user has already submitted before adding them as a reviewer
1 parent 038fd49 commit a5036b1

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/common/helper.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -357,7 +357,7 @@ async function validateDuplicate (modelName, queryParams, errorMessage) {
357357
*/
358358
async function getRequest (url, query) {
359359
const m2mToken = await m2m.getMachineToken(config.AUTH0_CLIENT_ID, config.AUTH0_CLIENT_SECRET)
360-
360+
logger.debug(`GET ${url} with query ${JSON.stringify(query)}`);
361361
return request
362362
.get(url)
363363
.set('Authorization', `Bearer ${m2mToken}`)

src/services/ResourceService.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,8 @@ async function init (currentUser, challengeId, resource, isCreated) {
295295
if (isCreated && (resource.roleId === config.REVIEWER_RESOURCE_ROLE_ID || resource.roleId === config.ITERATIVE_REVIEWER_RESOURCE_ROLE_ID)) {
296296
const submissionsRes = await helper.getRequest(`${config.SUBMISSIONS_API_URL}`, { challengeId: challengeId, perPage: 100, memberId: memberId })
297297
const submissions = submissionsRes.body
298-
if (submissions.length !== 0) {
298+
logger.debug(`Submissions made by member ${memberId} in challenge ${challengeId}: ${JSON.stringify(submissions)}`)
299+
if (submissions.meta.totalCount !== 0) {
299300
throw new errors.ConflictError(`The member has already submitted to the challenge and cannot have a Reviewer or Iterative Reviewer role`)
300301
}
301302
}

0 commit comments

Comments
 (0)