changed to completed by

Author: stevomcallister

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/models/hmpps/ReviewSchedule.kt

@@ -29,8 +29,8 @@ data class ReviewSchedule(
   val updatedAt: Instant,
   val updatedAtPrison: String,
   val version: Int,
-  var conductedBy: String?,
-  var conductedRole: String?,
+  var completedBy: String?,
+  var completedByRole: String?,
 )
 
 data class ActionPlanReviewsResponse(

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/services/GetReviewScheduleForPersonService.kt

@@ -34,15 +34,17 @@ class GetReviewScheduleForPersonService(
 
     // Step 4: Update review schedules with completed reviews
     val updatedReviewSchedules =
-      reviewSchedulesResponse.data.reviewSchedules.map { reviewSchedule ->
-        val completed = mappedReviews[reviewSchedule.reference]
-        completed?.let {
-          reviewSchedule.copy(
-            conductedBy = it.conductedBy,
-            conductedRole = it.conductedByRole,
-          )
-        } ?: reviewSchedule
-      }
+      reviewSchedulesResponse.data.reviewSchedules
+        .filter { it.status == "COMPLETED" }
+        .map { reviewSchedule ->
+          val completed = mappedReviews[reviewSchedule.reference]
+          completed?.let {
+            reviewSchedule.copy(
+              completedBy = it.conductedBy,
+              completedByRole = it.conductedByRole,
+            )
+          } ?: reviewSchedule
+        }
 
     // Step 5: Return the updated review schedules
     return Response(