making project compatible with current connect app so that the service can be deployed without front end changes

Author: Vikas Agarwal

src/events/projects/index.js

@@ -34,8 +34,10 @@ const indexProject = Promise.coroutine(function* (logger, msg) { // eslint-disab
       const detail = _.find(memberDetails, md => md.userId === single.userId);
       return _.merge(single, _.pick(detail, 'handle', 'firstName', 'lastName', 'email'));
     });
-    // update project member record with details
-    data.phases = data.phases.map(phase => _.omit(phase, ['deletedAt', 'deletedBy']));
+    if (data.phases) {
+      // removes non required fields from phase objects
+      data.phases = data.phases.map(phase => _.omit(phase, ['deletedAt', 'deletedBy']));
+    }
     // add the record to the index
     const result = yield eClient.index({
       index: ES_PROJECT_INDEX,

src/routes/admin/project-index-create.js

@@ -57,8 +57,10 @@ module.exports = [
         if (!project) {
           return Promise.resolve(null);
         }
-        // removs the delete audit fields from the index data
-        project.phases = project.phases.map(phase => _.omit(phase, ['deletedAt', 'deletedBy']));
+        if (project.phases) {
+          // removs the delete audit fields from the index data
+          project.phases = project.phases.map(phase => _.omit(phase, ['deletedAt', 'deletedBy']));
+        }
         return models.ProjectMember.getActiveProjectMembers(project.id)
         .then((currentProjectMembers) => {
           // check context for project members

src/routes/projects/create.js

@@ -81,6 +81,7 @@ function createProjectAndPhases(req, project, projectTemplate, productTemplates)
   }).then((newProject) => {
     result.newProject = newProject;
 
+    // backward compatibility for releasing the service before releasing the front end
     if (!projectTemplate) {
       return Promise.resolve(result);
     }
@@ -132,6 +133,9 @@ function createProjectAndPhases(req, project, projectTemplate, productTemplates)
  * @returns {Promise} the promise that resolves to an object containing validated project and product templates
  */
 function validateAndFetchTemplates(templateId) {
+  // backward compatibility for releasing the service before releasing the front end
+  // we ignore missing template id field and create a project without phase/products
+  if (!templateId) return Promise.resolve(null);
   return models.ProjectTemplate.findById(templateId, { raw: true })
   .then((existingProjectTemplate) => {
     if (!existingProjectTemplate) {