From ef86948eb810a0f91ce79897ad53eba7faf1a65a Mon Sep 17 00:00:00 2001 From: RishiRaj Date: Wed, 16 Jan 2019 16:37:26 +0530 Subject: [PATCH 1/2] Added debug log. --- src/routes/projectMemberInvites/create.js | 1 + 1 file changed, 1 insertion(+) diff --git a/src/routes/projectMemberInvites/create.js b/src/routes/projectMemberInvites/create.js index 7717bdc9..2f2b75e2 100644 --- a/src/routes/projectMemberInvites/create.js +++ b/src/routes/projectMemberInvites/create.js @@ -207,6 +207,7 @@ module.exports = [ const forbidUserList = []; _.zip(invite.userIds, rolesList).forEach((data) => { const [userId, roles] = data; + req.log.debug(roles); if (!util.hasIntersection(MANAGER_ROLES, roles)) { forbidUserList.push(userId); From edb66c889dac9905d5f7f5801f6beb62e9cfbd07 Mon Sep 17 00:00:00 2001 From: RishiRaj Date: Wed, 16 Jan 2019 17:26:50 +0530 Subject: [PATCH 2/2] updated check for adding user as observer --- src/routes/projectMemberInvites/create.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/routes/projectMemberInvites/create.js b/src/routes/projectMemberInvites/create.js index 2f2b75e2..3214c493 100644 --- a/src/routes/projectMemberInvites/create.js +++ b/src/routes/projectMemberInvites/create.js @@ -181,7 +181,7 @@ module.exports = [ // permission: // user has to have constants.MANAGER_ROLES role // to be invited as PROJECT_MEMBER_ROLE.MANAGER - if (invite.role === PROJECT_MEMBER_ROLE.MANAGER) { + if (_.includes(PROJECT_MEMBER_MANAGER_ROLES, invite.role)) { _.forEach(invite.userIds, (userId) => { req.log.info(userId); promises.push(util.getUserRoles(userId, req.log, req.id));