Merge pull request #773 from topcoder-platform/develop

Fix up invitation by email

Author: jmgasper

.circleci/config.yml

@@ -1,13 +1,12 @@
 version: 2.1
 python_env: &python_env
   docker:
-    - image: cimg/python:3.11.0-browsers
+    - image: cimg/python:3.11.7-browsers
 install_awscli: &install_awscli
   name: "Install awscli"
   command: |
     sudo apt update
-    sudo apt install jq
-    sudo apt install python3-pip
+    sudo apt install jq python3-pip
     sudo pip3 install awscli --upgrade
 install_deploysuite: &install_deploysuite
   name: Installation of install_deploysuite.

config/development.json

@@ -5,7 +5,7 @@
   "connectProjectsUrl": "https://connect.topcoder-dev.com/projects/",
   "fileServiceEndpoint": "https://api.topcoder-dev.com/v5/files",
   "memberServiceEndpoint": "https://api.topcoder-dev.com/v5/members",
-  "identityServiceEndpoint": "https://api.topcoder-dev.com/v3/",
+  "identityServiceEndpoint": "https://api.topcoder-dev.com/v5/",
   "taasJobApiUrl": "https://api.topcoder-dev.com/v5/jobs",
   "sfdcBillingAccountNameField": "Billing_Account_Name__c",
   "sfdcBillingAccountMarkupField": "Mark_Up__c",

package-lock.json

@@ -294,8 +294,13 @@ module.exports = [
     // whom we are inviting, because Member Service has a loose search logic and may return
     // users with handles whom we didn't search for
       .then((foundUsers) => {
-        const lowerCaseHandles = invite.handles.map(handle => handle.toLowerCase());
-        return foundUsers.filter(foundUser => _.includes(lowerCaseHandles, foundUser.handleLower));
+        if(invite.handles) {
+          const lowerCaseHandles = invite.handles.map(handle => handle.toLowerCase());
+          return foundUsers.filter(foundUser => _.includes(lowerCaseHandles, foundUser.handleLower));
+        }
+        else {
+          return []
+        }
       })
       .then((inviteUsers) => {
         const members = req.context.currentProjectMembers;
@@ -436,6 +441,7 @@ module.exports = [
             }
           });
       }).catch((err) => {
+        console.log(err)
         if (failed.length) {
           res.status(403).json(_.assign({}, { success: [] }, { failed }));
         } else next(err);