[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, **/*.jpg, **/*.png, **/*.jpeg, **/*.bmp, **/*.webp" # 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

@@ -20,7 +20,8 @@ import {
   UPDATE_PROJECT_FAILURE,
   ADD_PROJECT_ATTACHMENT_SUCCESS,
   UPDATE_PROJECT_ATTACHMENT_SUCCESS,
-  REMOVE_PROJECT_ATTACHMENT_SUCCESS
+  REMOVE_PROJECT_ATTACHMENT_SUCCESS,
+  LOAD_PROJECT_INVITES
 } from '../config/constants'
 import {
   fetchProjectById,
@@ -30,9 +31,10 @@ import {
   createProjectApi,
   fetchBillingAccounts,
   fetchMemberProjects,
-  updateProjectApi
+  updateProjectApi,
+  getProjectInvites
 } from '../services/projects'
-import { checkAdmin } from '../util/tc'
+import { checkAdmin, checkManager } from '../util/tc'
 
 function _loadProjects (projectNameOrIdFilter = '', paramFilters = {}) {
   return (dispatch, getState) => {
@@ -54,7 +56,7 @@ function _loadProjects (projectNameOrIdFilter = '', paramFilters = {}) {
       }
     }
 
-    if (!checkAdmin(getState().auth.token)) {
+    if (!checkAdmin(getState().auth.token) && !checkManager(getState().auth.token)) {
       filters['memberOnly'] = true
     }
 
@@ -171,6 +173,18 @@ export function loadProjectTypes () {
   }
 }
 
+/**
+ * Loads project invites
+ */
+export function loadProjectInvites (projectId) {
+  return (dispatch) => {
+    return dispatch({
+      type: LOAD_PROJECT_INVITES,
+      payload: getProjectInvites(projectId)
+    })
+  }
+}
+
 /**
  * Creates a project
  */

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/ProjectCard/index.js

@@ -8,11 +8,11 @@ import { PROJECT_STATUSES } from '../../config/constants'
 
 import styles from './ProjectCard.module.scss'
 
-const ProjectCard = ({ projectName, projectStatus, projectId, selected }) => {
+const ProjectCard = ({ projectName, projectStatus, projectId, selected, isInvited }) => {
   return (
     <div className={styles.container}>
       <Link
-        to={`/projects/${projectId}/challenges`}
+        to={`/projects/${projectId}/${isInvited ? 'invitation' : 'challenges'}`}
         className={cn(styles.projectName, { [styles.selected]: selected })}
       >
         <div className={styles.name}>
@@ -28,6 +28,7 @@ ProjectCard.propTypes = {
   projectStatus: PT.string.isRequired,
   projectId: PT.number.isRequired,
   projectName: PT.string.isRequired,
+  isInvited: PT.bool.isRequired,
   selected: PT.bool
 }
 

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,