Merge pull request #2433 from appirio-tech/issue-2410

Fix issue 2410: Enforce minimum design selection at checkpoint review milestone

Author: gondzo

src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx

@@ -40,6 +40,7 @@ class MilestoneTypeCheckpointReview extends React.Component {
     this.completeReview = this.completeReview.bind(this)
     this.toggleRejectedSection = this.toggleRejectedSection.bind(this)
     this.moveToReviewingState = this.moveToReviewingState.bind(this)
+    this.shouldDisableCompleteReviewButton = this.shouldDisableCompleteReviewButton.bind(this)
   }
 
   getLinksForReview() {
@@ -180,6 +181,18 @@ class MilestoneTypeCheckpointReview extends React.Component {
     return milestone[`${milestone.status}Text`]
   }
 
+  shouldDisableCompleteReviewButton(links, selectedLinks) {
+    const linksLength = links.length
+    const selectedLinksLength = selectedLinks.length
+    if (linksLength < 5 && selectedLinksLength === linksLength) {
+      return false
+    } else if (linksLength >= 5 && selectedLinksLength >= 5) {
+      return false
+    } else {
+      return true
+    }
+  }
+
   render() {
     const {
       milestone,
@@ -361,7 +374,7 @@ class MilestoneTypeCheckpointReview extends React.Component {
                     <button
                       className={'tc-btn tc-btn-primary'}
                       onClick={!currentUser.isCustomer ? this.showCompleteReviewConfirmation : this.completeReview}
-                      disabled={!isInReview}
+                      disabled={this.shouldDisableCompleteReviewButton(links, selectedLinks) && !isInReview}
                     >
                       Complete review ({
                         daysLeft >= 0