Skip to content
Snippets Groups Projects
Commit d117a605 authored by Sebastian Mihalache's avatar Sebastian Mihalache :hammer_pick:
Browse files

fix(publons): update parameter isPublons

parent 5b0b887c
No related branches found
No related tags found
2 merge requests!110Sprint 21 Features,!91Hin 904 fixes after merge
const logger = require('@pubsweet/logger') const logger = require('@pubsweet/logger')
const { const {
Team, Team,
User, User,
...@@ -124,8 +125,13 @@ module.exports = models => async (req, res) => { ...@@ -124,8 +125,13 @@ module.exports = models => async (req, res) => {
const userData = req.body const userData = req.body
const { firstName, lastName, isPublons } = userData const { firstName, lastName, isPublons } = userData
if (!services.checkForUndefinedParams(firstName, lastName)) {
return res
.status(400)
.json({ error: 'First name and last name are required.' })
}
if (process.env.PUBLONS_MOCK_EMAIL && isPublons) { if (isPublons && process.env.PUBLONS_MOCK_EMAIL) {
const mockEmail = process.env.PUBLONS_MOCK_EMAIL const mockEmail = process.env.PUBLONS_MOCK_EMAIL
userData.email = mockEmail.replace('__NAME__', `${firstName}.${lastName}`) userData.email = mockEmail.replace('__NAME__', `${firstName}.${lastName}`)
} }
......
...@@ -433,7 +433,7 @@ export default compose( ...@@ -433,7 +433,7 @@ export default compose(
reviewerData: values, reviewerData: values,
fragmentId: fragment.id, fragmentId: fragment.id,
collectionId: collection.id, collectionId: collection.id,
isPublon: true, isPublons: true,
}) })
.then(() => { .then(() => {
setFetching(false) setFetching(false)
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment