Conversation
| ['id', 'username', 'email', 'mobile', 'fullname'] | ||
| ); | ||
| req.user = userDetails; | ||
| req.user = lodash.pick(user, ['id', 'username', 'email', 'mobile', 'fullname', 'avatarPath']); |
There was a problem hiding this comment.
Expected indentation of 8 spaces but found 10 indent
|
|
||
| export { | ||
| sendSMS, sendMail, handleError, handleSuccess, uploadPictureLocally, uploadPictureToCloudinary, updateUserDetails | ||
| }; |
There was a problem hiding this comment.
Unexpected line break before this closing brace object-curly-newline
| } | ||
|
|
||
| export { | ||
| sendSMS, sendMail, handleError, handleSuccess, uploadPictureLocally, uploadPictureToCloudinary, updateUserDetails |
There was a problem hiding this comment.
Line 215 exceeds the maximum line length of 100 max-len
| }); | ||
| } | ||
|
|
||
| export { |
There was a problem hiding this comment.
Unexpected line break after this opening brace object-curly-newline
|
|
||
| return handleError(err, res); | ||
| }); | ||
| } |
|
|
||
| return user.update(fieldsInLowercase); | ||
| }) | ||
| .then((updatedUser) => { |
There was a problem hiding this comment.
Unexpected block statement surrounding arrow body; move the returned value immediately after the => arrow-body-style
| models.User.findOne({where: {username: req.user.username}}) | ||
| .then(user => { | ||
| if (!user) { | ||
| return Promise.reject('Something went wrong'); |
There was a problem hiding this comment.
Expected the Promise rejection reason to be an Error prefer-promise-reject-errors
| */ | ||
| const updateUserDetails = (req, res, fields) => { | ||
| models.User.findOne({where: {username: req.user.username}}) | ||
| .then(user => { |
There was a problem hiding this comment.
Expected parentheses around arrow function argument having a body with curly braces arrow-parens
| * @return {void} | ||
| */ | ||
| const updateUserDetails = (req, res, fields) => { | ||
| models.User.findOne({where: {username: req.user.username}}) |
There was a problem hiding this comment.
A space is required after '{' object-curly-spacing
A space is required before '}' object-curly-spacing
| height: 200 | ||
| }); | ||
| }) | ||
| } |
johadi
force-pushed
the
develop
branch
2 times, most recently
from
2fda31c to
f8cb6c4
Compare
2 participants
add docker