From 9584ac68e4213f68603f10aae6576c9ebc5e1f09 Mon Sep 17 00:00:00 2001
From: jomadoye <jedidiah.omadoye@andela.com>
Date: Sat, 25 Aug 2018 02:18:30 +0100
Subject: [PATCH 1/2] Fix issue 2410

---
 .../MilestoneTypeCheckpointReview.jsx             | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx b/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx
index 836688af1..c34c0d93d 100644
--- a/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx
+++ b/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.shouldActivateCompleteReviewButton = this.shouldActivateCompleteReviewButton.bind(this)
   }
 
   showCompleteReviewConfirmation() {
@@ -170,6 +171,18 @@ class MilestoneTypeCheckpointReview extends React.Component {
     return milestone[`${milestone.status}Text`]
   }
 
+  shouldActivateCompleteReviewButton(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,
@@ -346,7 +359,7 @@ class MilestoneTypeCheckpointReview extends React.Component {
                     <button
                       className={'tc-btn tc-btn-primary'}
                       onClick={!currentUser.isCustomer ? this.showCompleteReviewConfirmation : this.completeReview}
-                      disabled={!isInReview}
+                      disabled={this.shouldActivateCompleteReviewButton(links, selectedLinks) || !isInReview}
                     >
                       Complete review ({
                         daysLeft >= 0

From 79e62740bf3b072d435a221647765575582bf493 Mon Sep 17 00:00:00 2001
From: gondzo <gondzo@gmail.com>
Date: Wed, 29 Aug 2018 01:20:01 +0200
Subject: [PATCH 2/2] rename methods

---
 .../MilestoneTypeCheckpointReview.jsx                       | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx b/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx
index c34c0d93d..855345d60 100644
--- a/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx
+++ b/src/projects/detail/components/timeline/milestones/MilestoneTypeCheckpointReview/MilestoneTypeCheckpointReview.jsx
@@ -40,7 +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.shouldActivateCompleteReviewButton = this.shouldActivateCompleteReviewButton.bind(this)
+    this.shouldDisableCompleteReviewButton = this.shouldDisableCompleteReviewButton.bind(this)
   }
 
   showCompleteReviewConfirmation() {
@@ -171,7 +171,7 @@ class MilestoneTypeCheckpointReview extends React.Component {
     return milestone[`${milestone.status}Text`]
   }
 
-  shouldActivateCompleteReviewButton(links, selectedLinks) {
+  shouldDisableCompleteReviewButton(links, selectedLinks) {
     const linksLength = links.length
     const selectedLinksLength = selectedLinks.length
     if (linksLength < 5 && selectedLinksLength === linksLength) {
@@ -359,7 +359,7 @@ class MilestoneTypeCheckpointReview extends React.Component {
                     <button
                       className={'tc-btn tc-btn-primary'}
                       onClick={!currentUser.isCustomer ? this.showCompleteReviewConfirmation : this.completeReview}
-                      disabled={this.shouldActivateCompleteReviewButton(links, selectedLinks) || !isInReview}
+                      disabled={this.shouldDisableCompleteReviewButton(links, selectedLinks) && !isInReview}
                     >
                       Complete review ({
                         daysLeft >= 0