feat: respond to po comments

app/models/form/lettings/pages/new_build_handover_date.rb

@@ -1,11 +0,0 @@
-class Form::Lettings::Pages::NewBuildHandoverDate < ::Form::Page
-  def initialize(id, hsh, subsection)
-    super
-    @id = "new_build_handover_date"
-    @depends_on = [{ "is_renewal?" => false, "has_first_let_vacancy_reason?" => true }]
-  end
-
-  def questions
-    @questions ||= [Form::Lettings::Questions::VoiddateNewBuild.new(nil, nil, self)]
-  end
-end

app/models/form/lettings/pages/void_date.rb

@@ -2,7 +2,8 @@ class Form::Lettings::Pages::VoidDate < ::Form::Page
   def initialize(id, hsh, subsection)
     super
     @id = "void_date"
-    @depends_on = [{ "is_renewal?" => false, "vacancy_reason_not_renewal_or_first_let?" => true }]
+    @depends_on = [{ "is_renewal?" => false, "vacancy_reason_not_renewal_or_first_let?" => true },
+                   { "is_renewal?" => false, "has_first_let_vacancy_reason?" => true }]
   end
 
   def questions

app/models/form/lettings/questions/voiddate_new_build.rb

@@ -1,12 +0,0 @@
-class Form::Lettings::Questions::VoiddateNewBuild < ::Form::Question
-  def initialize(id, hsh, page)
-    super
-    @id = "voiddate"
-    @check_answer_label = "New-build handover date"
-    @header = "What is the new-build handover date?"
-    @type = "date"
-    @check_answers_card_number = 0
-    @hint_text = ""
-    @question_number = 23
-  end
-end

app/models/form/lettings/subsections/property_information.rb

@@ -22,7 +22,6 @@ class Form::Lettings::Subsections::PropertyInformation < ::Form::Subsection
       Form::Lettings::Pages::PropertyNumberOfBedrooms.new(nil, nil, self),
       Form::Lettings::Pages::VoidDate.new(nil, nil, self),
       Form::Lettings::Pages::VoidDateValueCheck.new(nil, nil, self),
-      Form::Lettings::Pages::NewBuildHandoverDate.new(nil, nil, self),
       Form::Lettings::Pages::PropertyMajorRepairs.new(nil, nil, self),
       Form::Lettings::Pages::PropertyMajorRepairsValueCheck.new(nil, nil, self),
     ].flatten.compact

config/forms/2022_2023.json

@@ -767,6 +767,18 @@
                 {
                   "renewal": 0,
                   "rsnvac": 19
+                },
+                {
+                  "renewal": 0,
+                  "rsnvac": 15
+                },
+                {
+                  "renewal": 0,
+                  "rsnvac": 16
+                },
+                {
+                  "renewal": 0,
+                  "rsnvac": 17
                 }
               ]
             },
@@ -806,32 +818,6 @@
                 }
               }
             },
-            "new_build_handover_date": {
-              "header": "",
-              "description": "",
-              "questions": {
-                "voiddate": {
-                  "check_answer_label": "New-build handover date",
-                  "header": "What is the new-build handover date?",
-                  "hint_text": "",
-                  "type": "date"
-                }
-              },
-              "depends_on": [
-                {
-                  "renewal": 0,
-                  "rsnvac": 15
-                },
-                {
-                  "renewal": 0,
-                  "rsnvac": 16
-                },
-                {
-                  "renewal": 0,
-                  "rsnvac": 17
-                }
-              ]
-            },
             "property_major_repairs": {
               "header": "",
               "description": "",