From eff1ca6250902d302368e05ef431f0394d5fefe7 Mon Sep 17 00:00:00 2001
From: Paul Robert Lloyd <me+git@paulrobertlloyd.com>
Date: Thu, 3 Feb 2022 15:45:49 +0000
Subject: [PATCH] Enable section descriptions and add note about privacy notice

---
 README.md                              | 1 +
 app/models/form/section.rb             | 3 ++-
 app/views/case_logs/_tasklist.html.erb | 3 +++
 config/forms/2021_2022.json            | 1 +
 spec/fixtures/forms/2021_2022.json     | 1 +
 spec/models/form/section_spec.rb       | 4 ++++
 6 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 4fea5097e..f75b018c9 100644
--- a/README.md
+++ b/README.md
@@ -107,6 +107,7 @@ The JSON should follow the structure:
   "sections": {
     "[snake_case_section_name_string]": {
       "label": String,
+      "description": String,
       "subsections": {
         "[snake_case_subsection_name_string]": {
           "label": String,
diff --git a/app/models/form/section.rb b/app/models/form/section.rb
index 477fc9f18..b99325f90 100644
--- a/app/models/form/section.rb
+++ b/app/models/form/section.rb
@@ -1,9 +1,10 @@
 class Form::Section
-  attr_accessor :id, :label, :subsections, :form
+  attr_accessor :id, :label, :description, :subsections, :form
 
   def initialize(id, hsh, form)
     @id = id
     @label = hsh["label"]
+    @description = hsh["description"]
     @form = form
     @subsections = hsh["subsections"].map { |s_id, s| Form::Subsection.new(s_id, s, self) }
   end
diff --git a/app/views/case_logs/_tasklist.html.erb b/app/views/case_logs/_tasklist.html.erb
index 85a8d6349..8d20a0cee 100644
--- a/app/views/case_logs/_tasklist.html.erb
+++ b/app/views/case_logs/_tasklist.html.erb
@@ -6,6 +6,9 @@
           <%= section.label %>
         </span>
       </h2>
+      <% if section.description %>
+        <p class="govuk-body"><%= section.description.html_safe %></p>
+      <% end %>
       <ul class="app-task-list__items">
         <% section.subsections.map do |subsection| %>
           <% subsection_status = subsection.status(@case_log) %>
diff --git a/config/forms/2021_2022.json b/config/forms/2021_2022.json
index 89a7f9068..2c7ce0eb5 100644
--- a/config/forms/2021_2022.json
+++ b/config/forms/2021_2022.json
@@ -173,6 +173,7 @@
     },
     "household": {
       "label": "About the household",
+      "description": "Make sure the tenant has seen <a class=\"govuk-link\" href=\"/files/privacy-notice.pdf\">the Department for Levelling Up, Housing & Communities (DLUHC) privacy notice</a> before completing this section.",
       "subsections": {
         "household_characteristics": {
           "label": "Household characteristics",
diff --git a/spec/fixtures/forms/2021_2022.json b/spec/fixtures/forms/2021_2022.json
index 490009f70..dac2b94d3 100644
--- a/spec/fixtures/forms/2021_2022.json
+++ b/spec/fixtures/forms/2021_2022.json
@@ -3,6 +3,7 @@
     "sections": {
       "household": {
         "label": "About the household",
+        "description": "Make sure the tenant has seen the privacy notice.",
         "subsections": {
           "household_characteristics": {
             "label": "Household characteristics",
diff --git a/spec/models/form/section_spec.rb b/spec/models/form/section_spec.rb
index 2c9da7180..76ea23ca5 100644
--- a/spec/models/form/section_spec.rb
+++ b/spec/models/form/section_spec.rb
@@ -16,6 +16,10 @@ RSpec.describe Form::Section, type: :model do
     expect(section.label).to eq("About the household")
   end
 
+  it "has a description" do
+    expect(section.description).to eq("Make sure the tenant has seen the privacy notice.")
+  end
+
   it "has subsections" do
     expected_subsections = %w[household_characteristics household_needs]
     expect(section.subsections.map(&:id)).to eq(expected_subsections)