Merge pull request #795 from DFE-Digital/feature/transfer-dates-and-h…

…istory reasons updated with correct keys
DFE-Digital · Jul 10, 2024 · 3f0b616 · 3f0b616
2 parents 911a640 + 801050b
commit 3f0b616
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/Dfe.PrepareTransfers.Web/Pages/Projects/AcademyAndTrustInformation/Index.cshtml b/Dfe.PrepareTransfers.Web/Pages/Projects/AcademyAndTrustInformation/Index.cshtml
@@ -3,7 +3,7 @@
 @model Dfe.PrepareTransfers.Web.Pages.Projects.AcademyAndTrustInformation.Index
 
 @{
-    ViewBag.Title = "Trust information and project dates";
+    ViewBag.Title = "Trust information";
     Layout = "_Layout";
 }
 
@@ -18,7 +18,7 @@
             Project reference: @Model.ProjectReference
         </span>
         <h1 class="govuk-heading-l">
-            Trust information and project dates
+            Trust information
         </h1>
         <hr class="govuk-section-break govuk-section-break--l">
     </div>

diff --git a/Dfe.PrepareTransfers.Web/Pages/Projects/TransferDates/PreviousAdvisoryBoard.cshtml.cs b/Dfe.PrepareTransfers.Web/Pages/Projects/TransferDates/PreviousAdvisoryBoard.cshtml.cs
@@ -31,7 +31,7 @@ public async Task<IActionResult> OnGetAsync()
             {
                 AdvisoryBoardDate = new DateViewModel
                 {
-                    Date = DateViewModel.SplitDateIntoDayMonthYear(projectResult.Dates.Htb),
+                    Date = DateViewModel.SplitDateIntoDayMonthYear(projectResult.Dates.PreviousAdvisoryBoardDate),
                     UnknownDate = projectResult.Dates.HasHtbDate is false
                 }
             };

diff --git a/Dfe.PrepareTransfers.Web/Pages/Projects/TransferDates/Reason.cshtml.cs b/Dfe.PrepareTransfers.Web/Pages/Projects/TransferDates/Reason.cshtml.cs
@@ -24,14 +24,14 @@ public class Reason : CommonPageModel
 
         private readonly Dictionary<string, string> reasonMappings = new()
         {
-            { "faster-progress", "Project is progressing faster than expected" },
+            { "project-is-progressing-faster-than-expected", "Project is progressing faster than expected" },
             { "error-correction", "Correcting an error" },
             { "incoming-trust", "Incoming trust" },
             { "outgoing-trust", "Outgoing trust" },
             { "school", "School" },
-            { "local-authority", "LA (local authority)" },
+            { "la-(local-authority)", "LA (local authority)" },
             { "diocese", "Diocese" },
-            { "tupe", "TuPE (Transfer of Undertakings Protection of Employment rights)" },
+            { "tupe-(transfer-of-undertakings-protection-of-employment-rights)", "TuPE (Transfer of Undertakings Protection of Employment rights)" },
             { "pensions", "Pensions" },
             { "union", "Union" },
             { "negative-press-coverage", "Negative press coverage" },
@@ -42,7 +42,7 @@ public class Reason : CommonPageModel
             { "buildings", "Buildings" },
             { "legal-documents", "Legal documents" },
             { "voluntary-deferral", "Voluntary deferral" },
-            { "federation", "In a federation" }
+            { "in-a-federation", "In a federation" }
         };
 
         public Reason(IProjects projectsRepository)
@@ -88,7 +88,7 @@ public async Task<IActionResult> OnPostAsync()
 
         private List<ReasonChange> GetReasonOptions(bool isDateSooner)
         {
-            var soonerReasons = new List<string> { "faster-progress", "error-correction" };
+            var soonerReasons = new List<string> { "project-is-progressing-faster-than-expected", "error-correction" };
             var laterReasons = reasonMappings.Keys.Except(soonerReasons).ToList();
 
             return (isDateSooner ? soonerReasons : laterReasons)