[PROD RELEASE] - WorkManager Changes - Connect Decommission

.github/workflows/code_reviewer.yml

@@ -0,0 +1,22 @@
+name: AI PR Reviewer
+
+on:
+  pull_request:
+    types:
+      - opened
+      - synchronize
+permissions:
+  pull-requests: write
+jobs:
+  tc-ai-pr-review:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout Repo
+        uses: actions/checkout@v3
+
+      - name: TC AI PR Reviewer
+        uses: topcoder-platform/tc-ai-pr-reviewer@master
+        with:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # The GITHUB_TOKEN is there by default so you just need to keep it like it is and not necessarily need to add it as secret as it will throw an error. [More Details](https://docs.github.com/en/actions/security-guides/automatic-token-authentication#about-the-github_token-secret)
+          LAB45_API_KEY: ${{ secrets.LAB45_API_KEY }}
+          exclude: "**/*.json, **/*.md" # Optional: exclude patterns separated by commas

src/actions/challenges.js

@@ -57,7 +57,7 @@ import {
 } from '../config/constants'
 import { loadProject } from './projects'
 import { removeChallengeFromPhaseProduct, saveChallengeAsPhaseProduct } from '../services/projects'
-import { checkAdmin } from '../util/tc'
+import { checkAdmin, checkManager } from '../util/tc'
 
 /**
  * Member challenges related redux actions
@@ -159,7 +159,11 @@ export function loadChallengesByPage (
       filters['projectId'] = projectId
     } else if (_.isObject(projectId) && projectId.value > 0) {
       filters['projectId'] = projectId.value
-    } else if (!checkAdmin(getState().auth.token) && userId) {
+    } else if (
+      !checkAdmin(getState().auth.token) &&
+      !checkManager(getState().auth.token) &&
+      userId
+    ) {
       // Note that we only add the memberId field if *no* project ID is given,
       // so that the list of *all challenges shows only those that the member is on
       filters['memberId'] = userId

src/actions/projects.js

@@ -32,7 +32,7 @@ import {
   fetchMemberProjects,
   updateProjectApi
 } from '../services/projects'
-import { checkAdmin } from '../util/tc'
+import { checkAdmin, checkManager } from '../util/tc'
 
 function _loadProjects (projectNameOrIdFilter = '', paramFilters = {}) {
   return (dispatch, getState) => {
@@ -54,7 +54,7 @@ function _loadProjects (projectNameOrIdFilter = '', paramFilters = {}) {
       }
     }
 
-    if (!checkAdmin(getState().auth.token)) {
+    if (!checkAdmin(getState().auth.token) && !checkManager(getState().auth.token)) {
       filters['memberOnly'] = true
     }
 

src/actions/sidebar.js

@@ -11,7 +11,7 @@ import {
   UNLOAD_PROJECTS_SUCCESS,
   PROJECTS_PAGE_SIZE
 } from '../config/constants'
-import { checkAdmin } from '../util/tc'
+import { checkAdmin, checkManager } from '../util/tc'
 import _ from 'lodash'
 
 /**
@@ -50,7 +50,7 @@ export function loadProjects (filterProjectName = '', paramFilters = {}) {
       }
     }
 
-    if (!checkAdmin(getState().auth.token)) {
+    if (!checkAdmin(getState().auth.token) && !checkManager(getState().auth.token)) {
       filters['memberOnly'] = true
     }
 

src/actions/users.js

@@ -10,33 +10,55 @@ import {
   SEARCH_USER_PROJECTS_SUCCESS,
   SEARCH_USER_PROJECTS_FAILURE
 } from '../config/constants'
+import _ from 'lodash'
 
 /**
  * Loads projects of the authenticated user
  */
-export function loadAllUserProjects (isAdmin = true) {
-  return (dispatch) => {
+export function loadAllUserProjects (params, isAdmin = true, isManager = true) {
+  return (dispatch, getState) => {
     dispatch({
       type: LOAD_ALL_USER_PROJECTS_PENDING
     })
 
+    const state = getState().users
+
     const filters = {
       status: 'active',
-      sort: 'lastActivityAt desc'
+      sort: 'lastActivityAt desc',
+      perPage: 20,
+      ...params
     }
-    if (!isAdmin) {
+
+    if (!isAdmin && !isManager) {
       filters['memberOnly'] = true
     }
 
-    fetchMemberProjects(filters).then(({ projects }) => dispatch({
+    fetchMemberProjects(filters).then(({ projects, pagination }) => dispatch({
       type: LOAD_ALL_USER_PROJECTS_SUCCESS,
-      projects
+      projects: _.uniqBy((filters.page ? state.allUserProjects || [] : []).concat(projects), 'id'),
+      total: pagination.xTotal,
+      page: pagination.xPage
     })).catch(() => dispatch({
       type: LOAD_ALL_USER_PROJECTS_FAILURE
     }))
   }
 }
 
+export function loadNextProjects (isAdmin = true, isManager = true) {
+  return (dispatch, getState) => {
+    const { page, total, allUserProjects } = getState().users
+    if (allUserProjects.length >= total) {
+      return
+    }
+
+    loadAllUserProjects(_.assign({}, {
+      perPage: 20,
+      page: page + 1
+    }), isAdmin, isManager)(dispatch, getState)
+  }
+}
+
 /**
  * Filter projects of the authenticated user
  *

src/components/UserCard/index.js

@@ -1,3 +1,5 @@
+import _ from 'lodash'
+import moment from 'moment'
 import React, { Component } from 'react'
 import PropTypes from 'prop-types'
 import cn from 'classnames'
@@ -6,7 +8,6 @@ import { PROJECT_ROLES } from '../../config/constants'
 import PrimaryButton from '../Buttons/PrimaryButton'
 import AlertModal from '../Modal/AlertModal'
 import { updateProjectMemberRole } from '../../services/projects'
-import _ from 'lodash'
 
 const theme = {
   container: styles.modalContainer
@@ -58,7 +59,7 @@ class UserCard extends Component {
   }
 
   render () {
-    const { user, onRemoveClick, isEditable } = this.props
+    const { isInvite, user, onRemoveClick, isEditable } = this.props
     const showRadioButtons = _.includes(_.values(PROJECT_ROLES), user.role)
     return (
       <div>
@@ -90,76 +91,90 @@ class UserCard extends Component {
         )}
         <div className={styles.item}>
           <div className={cn(styles.col5)}>
-            {user.handle}
-          </div>
-          <div className={cn(styles.col5)}>
-            {showRadioButtons && (<div className={styles.tcRadioButton}>
-              <input
-                name={`user-${user.id}`}
-                type='radio'
-                id={`read-${user.id}`}
-                checked={user.role === PROJECT_ROLES.READ}
-                onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.READ)}
-              />
-              <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`read-${user.id}`}>
-                <div>
-                  Read
-                </div>
-                <input type='hidden' />
-              </label>
-            </div>)}
-          </div>
-          <div className={cn(styles.col5)}>
-            {showRadioButtons && (<div className={styles.tcRadioButton}>
-              <input
-                name={`user-${user.id}`}
-                type='radio'
-                id={`write-${user.id}`}
-                checked={user.role === PROJECT_ROLES.WRITE}
-                onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.WRITE)}
-              />
-              <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`write-${user.id}`}>
-                <div>
-                  Write
-                </div>
-                <input type='hidden' />
-              </label>
-            </div>)}
-          </div>
-          <div className={cn(styles.col5)}>
-            {showRadioButtons && (<div className={styles.tcRadioButton}>
-              <input
-                name={`user-${user.id}`}
-                type='radio'
-                id={`full-access-${user.id}`}
-                checked={user.role === PROJECT_ROLES.MANAGER}
-                onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.MANAGER)}
-              />
-              <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`full-access-${user.id}`}>
-                <div>
-                  Full Access
-                </div>
-                <input type='hidden' />
-              </label>
-            </div>)}
-          </div>
-          <div className={cn(styles.col5)}>
-            {showRadioButtons && (<div className={styles.tcRadioButton}>
-              <input
-                name={`user-${user.id}`}
-                type='radio'
-                id={`copilot-${user.id}`}
-                checked={user.role === PROJECT_ROLES.COPILOT}
-                onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.COPILOT)}
-              />
-              <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`copilot-${user.id}`}>
-                <div>
-                  Copilot
-                </div>
-                <input type='hidden' />
-              </label>
-            </div>)}
+            {isInvite ? user.email : user.handle}
           </div>
+          {!isInvite && (
+            <>
+              <div className={cn(styles.col5)}>
+                {showRadioButtons && (<div className={styles.tcRadioButton}>
+                  <input
+                    name={`user-${user.id}`}
+                    type='radio'
+                    id={`read-${user.id}`}
+                    checked={user.role === PROJECT_ROLES.READ}
+                    onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.READ)}
+                  />
+                  <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`read-${user.id}`}>
+                    <div>
+                      Read
+                    </div>
+                    <input type='hidden' />
+                  </label>
+                </div>)}
+              </div>
+              <div className={cn(styles.col5)}>
+                {showRadioButtons && (<div className={styles.tcRadioButton}>
+                  <input
+                    name={`user-${user.id}`}
+                    type='radio'
+                    id={`write-${user.id}`}
+                    checked={user.role === PROJECT_ROLES.WRITE}
+                    onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.WRITE)}
+                  />
+                  <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`write-${user.id}`}>
+                    <div>
+                      Write
+                    </div>
+                    <input type='hidden' />
+                  </label>
+                </div>)}
+              </div>
+              <div className={cn(styles.col5)}>
+                {showRadioButtons && (<div className={styles.tcRadioButton}>
+                  <input
+                    name={`user-${user.id}`}
+                    type='radio'
+                    id={`full-access-${user.id}`}
+                    checked={user.role === PROJECT_ROLES.MANAGER}
+                    onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.MANAGER)}
+                  />
+                  <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`full-access-${user.id}`}>
+                    <div>
+                      Full Access
+                    </div>
+                    <input type='hidden' />
+                  </label>
+                </div>)}
+              </div>
+              <div className={cn(styles.col5)}>
+                {showRadioButtons && (<div className={styles.tcRadioButton}>
+                  <input
+                    name={`user-${user.id}`}
+                    type='radio'
+                    id={`copilot-${user.id}`}
+                    checked={user.role === PROJECT_ROLES.COPILOT}
+                    onChange={(e) => e.target.checked && this.updatePermission(PROJECT_ROLES.COPILOT)}
+                  />
+                  <label className={cn({ [styles.isDisabled]: !isEditable })} htmlFor={`copilot-${user.id}`}>
+                    <div>
+                      Copilot
+                    </div>
+                    <input type='hidden' />
+                  </label>
+                </div>)}
+              </div>
+            </>
+          )}
+          {isInvite && (
+            <>
+              <div className={cn(styles.col5)} />
+              <div className={cn(styles.col5)}>
+                Invited {moment(user.createdAt).format('MMM D, YY')}
+              </div>
+              <div className={cn(styles.col5)} />
+              <div className={cn(styles.col5)} />
+            </>
+          )}
           {isEditable ? (<div className={cn(styles.col5)}>
             <PrimaryButton
               text={'Remove'}
@@ -173,6 +188,7 @@ class UserCard extends Component {
 }
 
 UserCard.propTypes = {
+  isInvite: PropTypes.bool,
   user: PropTypes.object,
   updateProjectNember: PropTypes.func.isRequired,
   onRemoveClick: PropTypes.func.isRequired,

src/components/Users/Users.module.scss

@@ -118,6 +118,12 @@
       text-decoration: none;
       font-size: 12px;
     }
+
+    &.inviteEmailInput {
+      input {
+        width: 250px;
+      }
+    }
   }
 }
 
@@ -395,10 +401,15 @@
 }
 
 .addButtonContainer {
-  width: 110px;
+  display: flex;
+  justify-content: flex-start;
   height: 30px;
   margin-top: 20px;
   margin-bottom: 20px;
+  gap: 8px;
+  > * {
+    width: 125px;
+  }
 }
 
 .addUserContentContainer {