Merge pull request #3325 from fikzzzy/feature/new-roles

vikasrohit · web-flow · commit 9cae21e525f5 · 2019-09-26T15:25:34.000+05:30
implement new project roles
diff --git a/src/components/TeamManagement/TopcoderManagementDialog.js b/src/components/TeamManagement/TopcoderManagementDialog.js
@@ -44,6 +44,18 @@ class TopcoderManagementDialog extends React.Component {
     }, {
       title: 'Account Manager',
       value: 'account_manager',
+    }, {
+      title: 'Account Executive',
+      value: 'account_executive',
+    }, {
+      title: 'Program Manager',
+      value: 'program_manager',
+    }, {
+      title: 'Solution Architect',
+      value: 'solution_architect',
+    }, {
+      title: 'Project Manager',
+      value: 'project_manager',
     }]
   }
 
@@ -208,7 +220,7 @@ class TopcoderManagementDialog extends React.Component {
                         </div>
                       )
                     }
-                    const types = ['Observer', 'Copilot', 'Manager', 'Account Manager']
+                    const types = ['Observer', 'Copilot', 'Manager', 'Account Manager', 'Account Executive', 'Program Manager', 'Solution Architect', 'Project Manager']
                     const currentType = role
                     const onClick = (type) => {
                       this.onUserRoleChange(member.userId, member.id, type)
diff --git a/src/components/TopBar/TopBarContainer.js b/src/components/TopBar/TopBarContainer.js
@@ -11,7 +11,12 @@ import {
   ROLE_ADMINISTRATOR,
   ROLE_CONNECT_ADMIN,
   ROLE_CONNECT_ACCOUNT_MANAGER,
-  DOMAIN
+  DOMAIN,
+  ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE,
+  ROLE_PRESALES,
+  ROLE_ACCOUNT_EXECUTIVE,
+  ROLE_PROJECT_MANAGER,
+  ROLE_PROGRAM_MANAGER, ROLE_SOLUTION_ARCHITECT
 } from '../../config/constants'
 import ConnectLogoMono from '../../assets/icons/connect-logo-mono.svg'
 import { getAvatarResized, getFullNameWithFallback } from '../../helpers/tcHelpers.js'
@@ -165,7 +170,20 @@ class TopBarContainer extends React.Component {
 
 const mapStateToProps = ({ loadUser }) => {
   let isPowerUser = false
-  const roles = [ROLE_CONNECT_COPILOT, ROLE_CONNECT_MANAGER, ROLE_CONNECT_ACCOUNT_MANAGER, ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN]
+  const roles = [
+    ROLE_CONNECT_COPILOT,
+    ROLE_CONNECT_MANAGER,
+    ROLE_CONNECT_ACCOUNT_MANAGER,
+    ROLE_ADMINISTRATOR,
+    ROLE_CONNECT_ADMIN,
+    
+    ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE,
+    ROLE_PRESALES,
+    ROLE_ACCOUNT_EXECUTIVE,
+    ROLE_PROJECT_MANAGER,
+    ROLE_PROGRAM_MANAGER,
+    ROLE_SOLUTION_ARCHITECT,
+  ]
   if (loadUser.user) {
     isPowerUser = loadUser.user.roles.some((role) => roles.indexOf(role) !== -1)
   }
diff --git a/src/config/constants.js b/src/config/constants.js
@@ -626,7 +626,10 @@ export const PROJECT_ROLE_ACCOUNT_MANAGER = 'account_manager'
 export const PROJECT_ROLE_CUSTOMER = 'customer'
 export const PROJECT_ROLE_OWNER = 'owner'
 export const PROJECT_ROLE_MEMBER = 'member' // this is need for notifications
-
+export const PROJECT_ROLE_ACCOUNT_EXECUTIVE = 'account_executive'
+export const PROJECT_ROLE_PROGRAM_MANAGER = 'program_manager'
+export const PROJECT_ROLE_SOLUTION_ARCHITECT = 'solution_architect'
+export const PROJECT_ROLE_PROJECT_MANAGER = 'project_manager'
 /*
  * Events
  */
@@ -642,6 +645,12 @@ export const ROLE_CONNECT_ACCOUNT_MANAGER = 'Connect Account Manager'
 export const ROLE_CONNECT_ADMIN = 'Connect Admin'
 export const ROLE_ADMINISTRATOR = 'administrator'
 export const ROLE_CONNECT_COPILOT_MANAGER = 'Connect Copilot Manager'
+export const ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE = 'Business Development Representative'
+export const ROLE_PRESALES = 'Presales'
+export const ROLE_ACCOUNT_EXECUTIVE = 'Account Executive'
+export const ROLE_PROGRAM_MANAGER = 'Program Manager'
+export const ROLE_SOLUTION_ARCHITECT = 'Solution Architect'
+export const ROLE_PROJECT_MANAGER = 'Project Manager'
 
 // FIXME .. remove defaults
 export const FILE_PICKER_API_KEY = process.env.FILE_PICKER_API_KEY || ''
diff --git a/src/config/permissions.js b/src/config/permissions.js
@@ -6,31 +6,98 @@ import {
   ROLE_ADMINISTRATOR,
   ROLE_CONNECT_ADMIN,
   ROLE_CONNECT_MANAGER,
-  ROLE_CONNECT_ACCOUNT_MANAGER
+  ROLE_CONNECT_ACCOUNT_MANAGER,
+  ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE,
+  ROLE_PRESALES,
+  ROLE_ACCOUNT_EXECUTIVE,
+  ROLE_PROGRAM_MANAGER,
+  ROLE_SOLUTION_ARCHITECT,
+  ROLE_PROJECT_MANAGER,
+  PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+  PROJECT_ROLE_PROJECT_MANAGER,
+  PROJECT_ROLE_PROGRAM_MANAGER,
+  PROJECT_ROLE_SOLUTION_ARCHITECT
 } from './constants'
 
 export default {
   EDIT_PROJECT_PLAN: {
-    projectRoles: [PROJECT_ROLE_MANAGER, PROJECT_ROLE_COPILOT, ROLE_CONNECT_ADMIN],
+    projectRoles: [
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT,
+      PROJECT_ROLE_COPILOT,
+      ROLE_CONNECT_ADMIN
+    ],
     topcoderRoles: [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN],
   },
   INVITE_TOPCODER_MEMBER: {
-    projectRoles: [PROJECT_ROLE_MANAGER, PROJECT_ROLE_ACCOUNT_MANAGER, ROLE_CONNECT_ADMIN],
-    topcoderRoles: [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN, ROLE_CONNECT_MANAGER, ROLE_CONNECT_ACCOUNT_MANAGER],
+    projectRoles: [
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_ACCOUNT_MANAGER,
+      PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT,
+      ROLE_CONNECT_ADMIN
+    ],
+    topcoderRoles: [
+      ROLE_ADMINISTRATOR,
+      ROLE_CONNECT_ADMIN,
+      ROLE_CONNECT_MANAGER,
+      ROLE_CONNECT_ACCOUNT_MANAGER,
+      ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE,
+      ROLE_PRESALES,
+      ROLE_ACCOUNT_EXECUTIVE,
+      ROLE_PROGRAM_MANAGER,
+      ROLE_SOLUTION_ARCHITECT,
+      ROLE_PROJECT_MANAGER,
+    ],
   },
   ACCESS_PRIVATE_POST: {
-    topcoderRoles: [PROJECT_ROLE_COPILOT, ROLE_CONNECT_MANAGER, ROLE_CONNECT_ADMIN, ROLE_ADMINISTRATOR]
+    topcoderRoles: [
+      PROJECT_ROLE_COPILOT,
+      ROLE_CONNECT_MANAGER,
+      ROLE_CONNECT_ADMIN,
+      ROLE_ADMINISTRATOR,
+      ROLE_PROGRAM_MANAGER,
+      ROLE_SOLUTION_ARCHITECT,
+      ROLE_PROJECT_MANAGER,
+    ]
   },
   ACCESS_BUDGET_REPORT: {
-    projectRoles: [PROJECT_ROLE_CUSTOMER, PROJECT_ROLE_MANAGER, PROJECT_ROLE_ACCOUNT_MANAGER],
+    projectRoles: [
+      PROJECT_ROLE_CUSTOMER,
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_ACCOUNT_MANAGER,
+      PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT
+    ],
     topcoderRoles: [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN],
   },
   ACCESS_BUDGET_SPENT_REPORT: {
-    projectRoles: [PROJECT_ROLE_MANAGER, PROJECT_ROLE_ACCOUNT_MANAGER],
+    projectRoles: [
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_ACCOUNT_MANAGER,
+      PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT
+    ],
     topcoderRoles: [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN],
   },
   ACCESS_INVOICE_REPORT: {
-    projectRoles: [PROJECT_ROLE_CUSTOMER, PROJECT_ROLE_MANAGER, PROJECT_ROLE_ACCOUNT_MANAGER],
+    projectRoles: [
+      PROJECT_ROLE_CUSTOMER,
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_ACCOUNT_MANAGER,
+      PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT
+    ],
     topcoderRoles: [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN],
   }
 }
diff --git a/src/projects/components/projectsCard/ProjectCardBody.jsx b/src/projects/components/projectsCard/ProjectCardBody.jsx
@@ -5,7 +5,13 @@ import { Link } from 'react-router-dom'
 import ProjectProgress from '../../../components/ProjectProgress/ProjectProgress'
 import ProjectStatus from '../../../components/ProjectStatus/ProjectStatus'
 import editableProjectStatus from '../../../components/ProjectStatus/editableProjectStatus'
-import { PROJECT_STATUS_ACTIVE, PROJECT_STATUS_COMPLETED, PROJECT_ROLE_COPILOT, PROJECT_ROLE_MANAGER } from '../../../config/constants'
+import {
+  PROJECT_STATUS_ACTIVE,
+  PROJECT_STATUS_COMPLETED,
+  PROJECT_ROLE_COPILOT,
+  PROJECT_ROLE_MANAGER,
+  PROJECT_ROLE_PROGRAM_MANAGER, PROJECT_ROLE_SOLUTION_ARCHITECT, PROJECT_ROLE_PROJECT_MANAGER
+} from '../../../config/constants'
 import './ProjectCardBody.scss'
 import _ from 'lodash'
 
@@ -17,7 +23,13 @@ function ProjectCardBody({ project, projectCanBeActive, duration, currentMemberR
 
   const canEdit = canEditStatus && (
     project.status !== PROJECT_STATUS_COMPLETED && (isSuperUser || (currentMemberRole
-    && (_.indexOf([PROJECT_ROLE_COPILOT, PROJECT_ROLE_MANAGER], currentMemberRole) > -1)))
+    && (_.indexOf([
+      PROJECT_ROLE_COPILOT,
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT,
+    ], currentMemberRole) > -1)))
   )
 
   const progress = _.get(process, 'percent', 0)
diff --git a/src/projects/detail/containers/ProjectInfoContainer.js b/src/projects/detail/containers/ProjectInfoContainer.js
@@ -10,9 +10,21 @@ import { loadDashboardFeeds, loadProjectMessages } from '../../actions/projectTo
 import { loadTopic } from '../../../actions/topics'
 import { loadProjectPlan } from '../../actions/projectPlan'
 import { setDuration, getProjectNavLinks } from '../../../helpers/projectHelper'
-import { PROJECT_ROLE_OWNER, PROJECT_ROLE_COPILOT, PROJECT_ROLE_MANAGER,
-  DIRECT_PROJECT_URL, SALESFORCE_PROJECT_LEAD_LINK, PROJECT_STATUS_CANCELLED,  PROJECT_STATUS_ACTIVE,
-  PROJECT_STATUS_COMPLETED, PHASE_STATUS_REVIEWED, PHASE_STATUS_ACTIVE } from '../../../config/constants'
+import {
+  PROJECT_ROLE_OWNER,
+  PROJECT_ROLE_COPILOT,
+  PROJECT_ROLE_MANAGER,
+  DIRECT_PROJECT_URL,
+  SALESFORCE_PROJECT_LEAD_LINK,
+  PROJECT_STATUS_CANCELLED,
+  PROJECT_STATUS_ACTIVE,
+  PROJECT_STATUS_COMPLETED,
+  PHASE_STATUS_REVIEWED,
+  PHASE_STATUS_ACTIVE,
+  PROJECT_ROLE_PROJECT_MANAGER,
+  PROJECT_ROLE_PROGRAM_MANAGER,
+  PROJECT_ROLE_SOLUTION_ARCHITECT,
+} from '../../../config/constants'
 import PERMISSIONS from '../../../config/permissions'
 import { checkPermission } from '../../../helpers/permissions'
 import Panel from '../../../components/Panel/Panel'
@@ -423,7 +435,13 @@ class ProjectInfoContainer extends React.Component {
       productsTimelines, isProjectProcessing, notifications } = this.props
     let directLinks = null
     // check if direct links need to be added
-    const isMemberOrCopilot = _.indexOf([PROJECT_ROLE_COPILOT, PROJECT_ROLE_MANAGER], currentMemberRole) > -1
+    const isMemberOrCopilot = _.indexOf([
+      PROJECT_ROLE_COPILOT,
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT
+    ], currentMemberRole) > -1
     if (isMemberOrCopilot || isSuperUser) {
       directLinks = []
       if (project.directProjectId) {
@@ -453,7 +471,13 @@ class ProjectInfoContainer extends React.Component {
 
     const canEdit = (
       project.status !== PROJECT_STATUS_COMPLETED && (isSuperUser || (currentMemberRole
-      && (_.indexOf([PROJECT_ROLE_COPILOT, PROJECT_ROLE_MANAGER], currentMemberRole) > -1)))
+      && (_.indexOf([
+        PROJECT_ROLE_COPILOT,
+        PROJECT_ROLE_MANAGER,
+        PROJECT_ROLE_PROJECT_MANAGER,
+        PROJECT_ROLE_PROGRAM_MANAGER,
+        PROJECT_ROLE_SOLUTION_ARCHITECT
+      ], currentMemberRole) > -1)))
     )
 
     const progress = _.get(process, 'percent', 0)
diff --git a/src/projects/detail/containers/ScopeAndSpecificationContainer.jsx b/src/projects/detail/containers/ScopeAndSpecificationContainer.jsx
@@ -24,7 +24,11 @@ import {
   PROJECT_ROLE_MANAGER,
   PROJECT_ROLE_ACCOUNT_MANAGER,
   PROJECT_FEED_TYPE_PRIMARY,
-  PROJECT_FEED_TYPE_MESSAGES
+  PROJECT_FEED_TYPE_MESSAGES,
+  PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+  PROJECT_ROLE_PROJECT_MANAGER,
+  PROJECT_ROLE_PROGRAM_MANAGER,
+  PROJECT_ROLE_SOLUTION_ARCHITECT
 } from '../../../config/constants'
 import {
   updateProject,
@@ -134,7 +138,14 @@ class SpecificationContainer extends Component {
     } = this.props
     const editPriv = isSuperUser ? isSuperUser : !!currentMemberRole
     const isCustomer = _.indexOf([PROJECT_ROLE_OWNER, PROJECT_ROLE_CUSTOMER], currentMemberRole) > -1
-    const isManager = currentMemberRole && [PROJECT_ROLE_MANAGER, PROJECT_ROLE_ACCOUNT_MANAGER].indexOf(currentMemberRole) > -1
+    const isManager = currentMemberRole && [
+      PROJECT_ROLE_MANAGER,
+      PROJECT_ROLE_ACCOUNT_MANAGER,
+      PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+      PROJECT_ROLE_PROJECT_MANAGER,
+      PROJECT_ROLE_PROGRAM_MANAGER,
+      PROJECT_ROLE_SOLUTION_ARCHITECT
+    ].indexOf(currentMemberRole) > -1
 
     const attachmentsStorePath = `${PROJECT_ATTACHMENTS_FOLDER}/${project.id}/`
 
diff --git a/src/projects/detail/containers/TeamManagementContainer.jsx b/src/projects/detail/containers/TeamManagementContainer.jsx
@@ -13,7 +13,15 @@ import {
   ROLE_CONNECT_COPILOT_MANAGER,
   ROLE_CONNECT_MANAGER,
   ROLE_CONNECT_ACCOUNT_MANAGER,
-  PROJECT_ROLE_ACCOUNT_MANAGER
+  PROJECT_ROLE_ACCOUNT_MANAGER,
+  ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE,
+  ROLE_PRESALES,
+  ROLE_ACCOUNT_EXECUTIVE,
+  ROLE_PROGRAM_MANAGER,
+  ROLE_PROJECT_MANAGER,
+  ROLE_SOLUTION_ARCHITECT,
+  PROJECT_ROLE_PROJECT_MANAGER,
+  PROJECT_ROLE_PROGRAM_MANAGER, PROJECT_ROLE_SOLUTION_ARCHITECT, PROJECT_ROLE_ACCOUNT_EXECUTIVE
 } from '../../../config/constants'
 import TeamManagement from '../../../components/TeamManagement/TeamManagement'
 import {
@@ -124,20 +132,20 @@ class TeamManagementContainer extends Component {
     return _.map(members, m => {
       if (!m.userId && !m.role) return m
       // map role
-      switch (m.role) {
-      case PROJECT_ROLE_COPILOT:
+      if (m.role === PROJECT_ROLE_COPILOT) {
         m.isCopilot = true
-        break
-      case PROJECT_ROLE_CUSTOMER:
+      } else if (m.role === PROJECT_ROLE_CUSTOMER) {
         m.isCustomer = true
         m.isPrimary = m.isPrimary || false
-        break
-      case PROJECT_ROLE_MANAGER:
+      } else if ([
+        PROJECT_ROLE_MANAGER,
+        PROJECT_ROLE_ACCOUNT_MANAGER,
+        PROJECT_ROLE_ACCOUNT_EXECUTIVE,
+        PROJECT_ROLE_PROJECT_MANAGER,
+        PROJECT_ROLE_PROGRAM_MANAGER,
+        PROJECT_ROLE_SOLUTION_ARCHITECT
+      ].includes(m.role)) {
         m.isManager = true
-        break
-      case PROJECT_ROLE_ACCOUNT_MANAGER:
-        m.isManager = true
-        break
       }
       return _.assign({}, m, {
         photoURL: ''
@@ -223,7 +231,8 @@ class TeamManagementContainer extends Component {
 const mapStateToProps = ({loadUser, members, projectState}) => {
   const adminRoles = [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN]
   const powerUserRoles = [ROLE_CONNECT_COPILOT, ROLE_CONNECT_MANAGER, ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN]
-  const managerRoles = [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN, ROLE_CONNECT_MANAGER]
+  const managerRoles = [ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN, ROLE_CONNECT_MANAGER, ROLE_PROGRAM_MANAGER, ROLE_PROJECT_MANAGER, ROLE_SOLUTION_ARCHITECT]
+  const accountManagerRoles = [ROLE_CONNECT_ACCOUNT_MANAGER, ROLE_BUSINESS_DEVELOPMENT_REPRESENTATIVE, ROLE_PRESALES, ROLE_ACCOUNT_EXECUTIVE]
   return {
     currentUser: {
       userId: parseInt(loadUser.user.id),
@@ -232,7 +241,7 @@ const mapStateToProps = ({loadUser, members, projectState}) => {
       isManager: loadUser.user.roles.some((role) => managerRoles.indexOf(role) !== -1),
       isCustomer: !loadUser.user.roles.some((role) => powerUserRoles.indexOf(role) !== -1),
       isCopilotManager: _.indexOf(loadUser.user.roles, ROLE_CONNECT_COPILOT_MANAGER) > -1,
-      isAccountManager: _.indexOf(loadUser.user.roles, ROLE_CONNECT_ACCOUNT_MANAGER) > -1,
+      isAccountManager: loadUser.user.roles.some((role) => accountManagerRoles.indexOf(role) !== -1),
     },
     allMembers: _.values(members.members),
     processingInvites: projectState.processingInvites,
diff --git a/src/routes/notifications/constants/notifications.js b/src/routes/notifications/constants/notifications.js
