From 0a3c635cd69308cc73d20485b05594a75655e419 Mon Sep 17 00:00:00 2001
From: RishiRaj <rishiraj.sahu@topcoder.com>
Date: Tue, 21 Jul 2020 22:04:01 +0530
Subject: [PATCH 1/3] Removed dependency on phaseStatus to change projectStatus
 gitIssue#4048

---
 .circleci/config.yml                          |  2 +-
 src/components/ProjectInfo/ProjectInfo.jsx    |  9 +---
 .../ProjectStatus/ProjectStatus.scss          | 16 -------
 .../ProjectStatus/editableProjectStatus.js    | 42 +++----------------
 .../projectsCard/ProjectCardBody.jsx          |  4 +-
 5 files changed, 9 insertions(+), 64 deletions(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 105dcca28..c213d067e 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -128,7 +128,7 @@ workflows:
             - build-dev
           filters:
             branches:
-              only: ['dev']
+              only: ['dev', 'feature/removed_dependency_on_phaseStatus_to_change_projectStatus']
 
       - deployTest01:
           context : org-global
diff --git a/src/components/ProjectInfo/ProjectInfo.jsx b/src/components/ProjectInfo/ProjectInfo.jsx
index 7786145a9..7228815a4 100644
--- a/src/components/ProjectInfo/ProjectInfo.jsx
+++ b/src/components/ProjectInfo/ProjectInfo.jsx
@@ -7,7 +7,7 @@ import DeleteProjectModal from './DeleteProjectModal'
 import ProjectCardBody from '../../projects/components/projectsCard/ProjectCardBody'
 import MobileExpandable from '../MobileExpandable/MobileExpandable'
 import MediaQuery from 'react-responsive'
-import { SCREEN_BREAKPOINT_MD, PROJECT_STATUS_ACTIVE, PHASE_STATUS_ACTIVE, PHASE_STATUS_REVIEWED, PROJECT_ROLE_OWNER, PROJECT_ROLE_CUSTOMER } from '../../config/constants'
+import { SCREEN_BREAKPOINT_MD, PROJECT_ROLE_OWNER, PROJECT_ROLE_CUSTOMER } from '../../config/constants'
 import ReviewProjectButton from '../../projects/detail/components/ReviewProjectButton'
 import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip'
 import { TOOLTIP_DEFAULT_DELAY } from '../../config/constants'
@@ -29,12 +29,6 @@ class ProjectInfo extends Component {
 
     const code = _.get(project, 'details.utm.code', '')
 
-    const hasReviewedOrActivePhases = !!_.find(phases, (phase) => _.includes([PHASE_STATUS_REVIEWED, PHASE_STATUS_ACTIVE], phase.status))
-    const isProjectActive = project.status === PROJECT_STATUS_ACTIVE
-    const isV3Project = project.version === 'v3'
-    const projectCanBeActive =  !isV3Project || (!isProjectActive && hasReviewedOrActivePhases) || isProjectActive
-
-
     // prepare review button
     const showReviewBtn = project.status === 'draft' &&
       _.indexOf([PROJECT_ROLE_OWNER, PROJECT_ROLE_CUSTOMER], currentMemberRole) > -1
@@ -108,7 +102,6 @@ class ProjectInfo extends Component {
               {matches => (
                 <ProjectCardBody
                   project={project}
-                  projectCanBeActive={projectCanBeActive}
                   currentMemberRole={currentMemberRole}
                   descLinesCount={
                     /* has to be not too big value here,
diff --git a/src/components/ProjectStatus/ProjectStatus.scss b/src/components/ProjectStatus/ProjectStatus.scss
index 38ee57c1a..7c54c3dbb 100644
--- a/src/components/ProjectStatus/ProjectStatus.scss
+++ b/src/components/ProjectStatus/ProjectStatus.scss
@@ -171,22 +171,6 @@
         &.active {
           background: $tc-gray-10;
         }
-
-        &.disabled  {
-          cursor: default;
-
-          svg {
-            filter: grayscale(1);
-          }
-
-          span {
-            color: $tc-gray-30;
-          }
-
-          &:hover {
-            background: transparent;
-          }
-        }
   
         svg {
           margin-top: 7px;
diff --git a/src/components/ProjectStatus/editableProjectStatus.js b/src/components/ProjectStatus/editableProjectStatus.js
index 68347116b..9c1b1eb3e 100644
--- a/src/components/ProjectStatus/editableProjectStatus.js
+++ b/src/components/ProjectStatus/editableProjectStatus.js
@@ -16,7 +16,7 @@ import {
 import CarretDownNormal9px from '../../assets/icons/arrow-9px-carret-down-normal.svg'
 
 
-const hocStatusDropdown = (CompositeComponent, statusList, projectCanBeActive) => {
+const hocStatusDropdown = (CompositeComponent, statusList) => {
   class StatusDropdown extends Component {
     shouldDropdownUp() {
       if (this.refs.dropdown) {
@@ -36,12 +36,6 @@ const hocStatusDropdown = (CompositeComponent, statusList, projectCanBeActive) =
         return null
       }
 
-      const activestatusList = statusList.map((status) => ({
-        ...status,
-        disabled: !projectCanBeActive && status.value === PROJECT_STATUS_ACTIVE,
-        toolTipMessage: (!projectCanBeActive && status.value === PROJECT_STATUS_ACTIVE) ? 'To activate project there should be at least one phase in "Planned" status. Please, check "Project Plan" tab.' : null,
-      }))
-
       this.shouldDropdownUp()
       return (
         <div className="project-status-dropdown" ref="dropdown">
@@ -64,28 +58,8 @@ const hocStatusDropdown = (CompositeComponent, statusList, projectCanBeActive) =
               <div className="status-header">Project Status</div>
               <ul>
                 {
-                  activestatusList.sort((a, b) => a.order - b.order).map((item) =>
-                    item.toolTipMessage ? (
-                      <Tooltip key={item.value} theme="light" tooltipDelay={TOOLTIP_DEFAULT_DELAY}>
-                        <div className="tooltip-target">
-                          <li>
-                            <a
-                              href="javascript:"
-                              className={cn('status-option', 'status-' + item.value, { active: item.value === status, disabled: item.disabled })}
-                              onClick={(e) => {
-                                if (!item.disabled)
-                                  onItemSelect(item.value, e)
-                              }}
-                            >
-                              <CompositeComponent status={item} showText />
-                            </a>
-                          </li>
-                        </div>
-                        <div className="tooltip-body">
-                          {item.toolTipMessage}
-                        </div>
-                      </Tooltip>
-                    ) : (
+                  statusList.sort((a, b) => a.order - b.order).map((item) =>
+                    (
                       <div key={item.value} className="tooltip-target">
                         <li>
                           <a
@@ -169,11 +143,11 @@ const editableProjectStatus = (CompositeComponent) => class extends Component {
 
   render() {
     const { showStatusChangeDialog, newStatus, statusChangeReason } = this.state
-    const { canEdit, projectCanBeActive, status } = this.props
+    const { canEdit, status } = this.props
     const PROJECT_STATUS_VALUES = this.getProjectStatusDropdownValues(status)
     const StatusDropdown = canEdit
-      ? enhanceDropdown(hocStatusDropdown(CompositeComponent, PROJECT_STATUS_VALUES, projectCanBeActive))
-      : hocStatusDropdown(CompositeComponent, PROJECT_STATUS_VALUES, projectCanBeActive)
+      ? enhanceDropdown(hocStatusDropdown(CompositeComponent, PROJECT_STATUS_VALUES))
+      : hocStatusDropdown(CompositeComponent, PROJECT_STATUS_VALUES)
     return (
       <div className={cn('EditableProjectStatus', {'modal-active': showStatusChangeDialog})}>
         <div className="modal-overlay" onClick={ this.hideStatusChangeDialog }/>
@@ -197,10 +171,6 @@ editableProjectStatus.propTypes = {
    * Boolean flag to control editability of the project status. It does not render the dropdown if it is not editable.
    */
   canEdit: PropTypes.bool,
-  /**
-   * Boolean flag to control if project status can be switched to active.
-   */
-  projectCanBeActive: PropTypes.bool,
   /**
    * String representing project status
    */
diff --git a/src/projects/components/projectsCard/ProjectCardBody.jsx b/src/projects/components/projectsCard/ProjectCardBody.jsx
index daf2dcbff..629ec8dc1 100644
--- a/src/projects/components/projectsCard/ProjectCardBody.jsx
+++ b/src/projects/components/projectsCard/ProjectCardBody.jsx
@@ -16,7 +16,7 @@ import _ from 'lodash'
 
 const EnhancedProjectStatus = editableProjectStatus(ProjectStatus)
 
-function ProjectCardBody({ project, projectCanBeActive, currentMemberRole, descLinesCount = 8,
+function ProjectCardBody({ project, currentMemberRole, descLinesCount = 8,
   onChangeStatus, isSuperUser, showLink, showLinkURL, canEditStatus = true, hideStatus, history }) {
   if (!project) return null
 
@@ -57,7 +57,6 @@ function ProjectCardBody({ project, projectCanBeActive, currentMemberRole, descL
         {(project.status !== PROJECT_STATUS_ACTIVE || progress === 0) &&
           <EnhancedProjectStatus
             status={project.status}
-            projectCanBeActive={projectCanBeActive}
             showText
             withoutLabel
             currentMemberRole={currentMemberRole}
@@ -81,7 +80,6 @@ ProjectCardBody.defaultTypes = {
 
 ProjectCardBody.propTypes = {
   project: PT.object.isRequired,
-  projectCanBeActive: PT.bool,
   currentMemberRole: PT.string,
   showLink: PT.bool,
   showLinkURL: PT.string,

From 3ab6264a9d2e6701749ec2d04194f30845e66130 Mon Sep 17 00:00:00 2001
From: RishiRaj <rishiraj.sahu@topcoder.com>
Date: Tue, 21 Jul 2020 22:09:50 +0530
Subject: [PATCH 2/3] lint fix

---
 src/components/ProjectInfo/ProjectInfo.jsx            | 2 +-
 src/components/ProjectStatus/editableProjectStatus.js | 3 ---
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/components/ProjectInfo/ProjectInfo.jsx b/src/components/ProjectInfo/ProjectInfo.jsx
index 7228815a4..8ee0eb1a7 100644
--- a/src/components/ProjectInfo/ProjectInfo.jsx
+++ b/src/components/ProjectInfo/ProjectInfo.jsx
@@ -24,7 +24,7 @@ class ProjectInfo extends Component {
 
   render() {
     const { project, currentMemberRole,
-      onChangeStatus, isSuperUser, phases, onSubmitForReview, isProjectProcessing,
+      onChangeStatus, isSuperUser, onSubmitForReview, isProjectProcessing,
       showDeleteConfirm, toggleProjectDelete, onConfirmDelete, projectTemplates } = this.props
 
     const code = _.get(project, 'details.utm.code', '')
diff --git a/src/components/ProjectStatus/editableProjectStatus.js b/src/components/ProjectStatus/editableProjectStatus.js
index 9c1b1eb3e..f98964d21 100644
--- a/src/components/ProjectStatus/editableProjectStatus.js
+++ b/src/components/ProjectStatus/editableProjectStatus.js
@@ -4,13 +4,10 @@ import ProjectStatusChangeConfirmation from './ProjectStatusChangeConfirmation'
 import cn from 'classnames'
 import _ from 'lodash'
 import enhanceDropdown from 'appirio-tech-react-components/components/Dropdown/enhanceDropdown'
-import Tooltip from 'appirio-tech-react-components/components/Tooltip/Tooltip'
 import {
   PROJECT_STATUS,
-  PROJECT_STATUS_ACTIVE,
   PROJECT_STATUS_COMPLETED,
   PROJECT_STATUS_CANCELLED,
-  TOOLTIP_DEFAULT_DELAY,
   PROJECT_STATUS_DRAFT
 } from '../../config/constants'
 import CarretDownNormal9px from '../../assets/icons/arrow-9px-carret-down-normal.svg'

From 89d03c04599f94bc6d29b6558e6873ce240320b7 Mon Sep 17 00:00:00 2001
From: RishiRaj <rishiraj.sahu@topcoder.com>
Date: Wed, 22 Jul 2020 19:25:23 +0530
Subject: [PATCH 3/3] removed feature branch from deployable.

---
 .circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index c213d067e..105dcca28 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -128,7 +128,7 @@ workflows:
             - build-dev
           filters:
             branches:
-              only: ['dev', 'feature/removed_dependency_on_phaseStatus_to_change_projectStatus']
+              only: ['dev']
 
       - deployTest01:
           context : org-global