fix customer issue

Author: nursoltan-s

src/projects/detail/components/SimplePlan/CreateSimplePlan/CreateSimplePlan.jsx

@@ -48,8 +48,8 @@ class CreateSimplePlan extends React.Component {
     } = this.props
     const onClickMilestonesTab = () => {}
 
-    const isUpdatable =
-      isCustomer ? project.status === PHASE_STATUS_IN_REVIEW
+    const isInReview =
+      isCustomer ?  (milestones.filter( x => x.status === PHASE_STATUS_IN_REVIEW).length > 0 && isProjectLive)
         : isProjectLive
 
     if (milestones.length === 0) {
@@ -88,7 +88,8 @@ class CreateSimplePlan extends React.Component {
             onApproveMilestones={onApproveMilestones}
             projectMembers={project.members}
             project={project}
-            isUpdatable={isUpdatable}
+            isUpdatable={isProjectLive && !isCustomer}
+            isInReview={isInReview}
             isCustomer={isCustomer}
           />
         </div>

src/projects/detail/components/SimplePlan/ManageMilestones/ManageMilestones.jsx

@@ -259,6 +259,7 @@ class ManageMilestones extends React.Component {
       onChangeMilestones,
       isUpdatable,
       isCustomer,
+      isInReview,
       project,
     } = this.props
 
@@ -338,13 +339,13 @@ class ManageMilestones extends React.Component {
                 <col style={{ width: '11%' }} />{/* END DATE */}
                 <col style={{ width: '10%' }} />{/* STATUS */}
                 <col style={{ width: '13%' }} />{/* COPILOTS */}
-                {(isUpdatable) && (<col style={{ width: '64px' }} />)}{/* ACTION */}
+                {(isUpdatable || isInReview) && (<col style={{ width: '64px' }} />)}{/* ACTION */}
               </colgroup>
               <thead>
                 <MilestoneHeaderRow
                   milestones={milestones}
                   onChangeMilestones={onChangeMilestones}
-                  isUpdatable={isUpdatable}
+                  isUpdatable={isUpdatable || isInReview}
                   hideCheckbox={hideCheckbox}
                 />
               </thead>
@@ -373,6 +374,7 @@ class ManageMilestones extends React.Component {
                       phaseMembers={milestone.members}
                       isApproving={milestonesInApproval.indexOf(milestone.id) !== -1}
                       hideCheckbox={hideCheckbox}
+                      isInReview={isInReview}
                     />,
                     ...this.renderChallengeTable(milestone)
                   ]
@@ -397,6 +399,7 @@ ManageMilestones.propTypes = {
   projectMembers: PT.arrayOf(PT.shape()),
   isUpdatable: PT.bool,
   isCustomer: PT.bool,
+  isInReview: PT.bool,
   project: PT.shape()
 }
 

src/projects/detail/components/SimplePlan/components/MilestoneRow/MilestoneRow.jsx

@@ -46,7 +46,8 @@ function MilestoneRow({
   disableDeleteAction,
   isCustomer,
   isApproving,
-  hideCheckbox
+  hideCheckbox,
+  isInReview
 }) {
   const isNeedApproval = milestone.status === PHASE_STATUS_IN_REVIEW
   const showApproval = isCustomer && isNeedApproval
@@ -356,7 +357,7 @@ function MilestoneRow({
         </td>
       )}
       {
-        (isCustomer && isUpdatable) && (
+        (isCustomer && isInReview) && (
           <td styleName="action">
             <div styleName="inline-menu approve">
               {
@@ -405,6 +406,7 @@ MilestoneRow.propTypes = {
   isCustomer: PT.bool,
   members: PT.object,
   hideCheckbox: PT.bool,
+  isInReview: PT.bool,
 }
 
 export default MilestoneRow