From f4066f5c145a3288c94843c672cb5b8361b50887 Mon Sep 17 00:00:00 2001
From: chrisala <chris.godwin.ala@gmail.com>
Date: Wed, 11 Dec 2024 10:31:38 +1100
Subject: [PATCH] label change and targets fixed to 2 decimal places #3369

---
 grails-app/assets/javascripts/services.js         | 3 ++-
 grails-app/views/organisation/_serviceTargets.gsp | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/grails-app/assets/javascripts/services.js b/grails-app/assets/javascripts/services.js
index bf37e74ef..748ea60e0 100644
--- a/grails-app/assets/javascripts/services.js
+++ b/grails-app/assets/javascripts/services.js
@@ -76,7 +76,8 @@ function OrganisationServicesViewModel(serviceIds, allServices, outputTargets, p
         target.targetDate = ko.observable().extend({simpleDate:false});
 
         target.periodTargets = _.map(periods, function (period) {
-            return {period: period.value, target: ko.observable(0)};
+            // TODO in the future, allowing a score to specifiy the number of decimal places would be useful
+            return {period: period.value, target: ko.observable(0).extend({numericString:2})};
         });
 
         function evaluateAndAssignAverage() {
diff --git a/grails-app/views/organisation/_serviceTargets.gsp b/grails-app/views/organisation/_serviceTargets.gsp
index eed391963..8ce480e9a 100644
--- a/grails-app/views/organisation/_serviceTargets.gsp
+++ b/grails-app/views/organisation/_serviceTargets.gsp
@@ -7,14 +7,14 @@
         <th class="index" rowspan="2"></th>
         <th class="service required" rowspan="2">${serviceName ?: "Service"}</th>
         <th class="score required" rowspan="2" style="width: 20px">Target measure</th>
-        <th class="budget-cell required" rowspan="2">Overall target (%) <g:if test="${totalHelpText}"> <fc:iconHelp> ${totalHelpText} </fc:iconHelp></g:if></th>
+        <th class="budget-cell required" rowspan="2">Overall % target <g:if test="${totalHelpText}"> <fc:iconHelp> ${totalHelpText} </fc:iconHelp></g:if></th>
         <g:if test="${showTargetDate}">
             <th class="target-date required" rowspan="2">
                 Delivery date <g:if test="${deliveryHelpText}"> <fc:iconHelp> ${deliveryHelpText} </fc:iconHelp> </g:if>
             </th>
         </g:if>
         <!-- ko if: periods && periods.length -->
-        <th data-bind="attr:{colspan:periods.length+1}">${periodTargetsLabel ?: " % Targets by date"}</th>
+        <th data-bind="attr:{colspan:periods.length+1}">${periodTargetsLabel ?: " % targets by date"}</th>
         <!-- /ko -->
     </tr>
     <!-- ko if: periods && periods.length -->