refactor and rename copy variables

app/models/form/sales/pages/estate_management_fee.rb

@@ -1,7 +1,7 @@
 class Form::Sales::Pages::EstateManagementFee < ::Form::Page
   def initialize(id, hsh, subsection)
     super
-    @copy_key = "sales.sale_information.management_fee"
+    @copy_key = "sales.sale_information.monthly_charges.shared_ownership.management_fee"
   end
 
   def questions

app/models/form/sales/pages/leasehold_charges.rb

@@ -8,9 +8,9 @@ class Form::Sales::Pages::LeaseholdCharges < ::Form::Page
     if form.start_year_2025_or_later?
       case @ownershipsch
       when 1
-        "sales.sale_information.leaseholdcharges.shared_ownership"
+        "sales.sale_information.monthly_charges.shared_ownership.service_charges"
       when 2
-        "sales.sale_information.leaseholdcharges.discounted_ownership"
+        "sales.sale_information.monthly_charges.discounted_ownership.leasehold_charges"
       end
     else
       "sales.sale_information.leaseholdcharges"

app/models/form/sales/questions/has_leasehold_charges.rb

@@ -22,9 +22,9 @@ class Form::Sales::Questions::HasLeaseholdCharges < ::Form::Question
     if form.start_year_2025_or_later?
       case @ownershipsch
       when 1
-        "sales.sale_information.leaseholdcharges.shared_ownership.has_mscharge"
+        "sales.sale_information.monthly_charges.shared_ownership.service_charges.has_mscharge"
       when 2
-        "sales.sale_information.leaseholdcharges.discounted_ownership.has_mscharge"
+        "sales.sale_information.monthly_charges.discounted_ownership.leasehold_charges.has_mscharge"
       end
     else
       "sales.sale_information.leaseholdcharges.has_mscharge"

app/models/form/sales/questions/has_management_fee.rb

@@ -2,7 +2,7 @@ class Form::Sales::Questions::HasManagementFee < ::Form::Question
   def initialize(id, hsh, subsection)
     super
     @id = "has_management_fee"
-    @copy_key = "sales.sale_information.management_fee.has_management_fee"
+    @copy_key = "sales.sale_information.monthly_charges.shared_ownership.management_fee.has_management_fee"
     @type = "radio"
     @answer_options = ANSWER_OPTIONS
     @conditional_for = {

app/models/form/sales/questions/leasehold_charges.rb

@@ -15,9 +15,9 @@ class Form::Sales::Questions::LeaseholdCharges < ::Form::Question
     if form.start_year_2025_or_later?
       case @ownershipsch
       when 1
-        "sales.sale_information.leaseholdcharges.shared_ownership.mscharge"
+        "sales.sale_information.monthly_charges.shared_ownership.service_charges.mscharge"
       when 2
-        "sales.sale_information.leaseholdcharges.discounted_ownership.mscharge"
+        "sales.sale_information.monthly_charges.discounted_ownership.leasehold_charges.mscharge"
       end
     else
       "sales.sale_information.leaseholdcharges.mscharge"

app/models/form/sales/questions/management_fee.rb

@@ -2,7 +2,7 @@ class Form::Sales::Questions::ManagementFee < ::Form::Question
   def initialize(id, hsh, subsection)
     super
     @id = "management_fee"
-    @copy_key = "sales.sale_information.management_fee.management_fee"
+    @copy_key = "sales.sale_information.monthly_charges.shared_ownership.management_fee.management_fee"
     @type = "numeric"
     @min = 1
     @step = 0.01

config/locales/forms/2025/sales/sale_information.en.yml

@@ -245,8 +245,9 @@ en:
               hint_text: "Amount paid before any charges"
               question_text: "What is the basic monthly rent after staircasing?"
 
-          leaseholdcharges:
+          monthly_charges:
             shared_ownership:
+              service_charges:
                 page_header: ""
                 has_mscharge:
                   check_answer_label: "Property service charges"
@@ -258,7 +259,20 @@ en:
                   check_answer_prompt: ""
                   hint_text: ""
                   question_text: "Enter the total monthly charge"
+              management_fee:
+                page_header: ""
+                has_management_fee:
+                  check_answer_label: "Monthly estate management fee"
+                  check_answer_prompt: "Tell us if the property has an estate management fee"
+                  hint_text: "Estate management fees are typically used for the maintenance of communal gardens, payments, private roads, car parks and/or play areas within new build estates."
+                  question_text: "Does the property have an estate management fee?"
+                management_fee:
+                  check_answer_label: "Monthly estate management fee"
+                  check_answer_prompt: ""
+                  hint_text: ""
+                  question_text: "Enter the total monthly management fee"
             discounted_ownership:
+              leasehold_charges:
                 page_header: ""
                 has_mscharge:
                   check_answer_label: "Property leasehold charges"
@@ -298,16 +312,3 @@ en:
             check_answer_prompt: ""
             hint_text: "For all schemes except Right to Buy (RTB), Preserved Right to Buy (PRTB), Voluntary Right to Buy (VRTB) and Rent to Buy"
             question_text: "What was the amount of any loan, grant, discount or subsidy given?"
-
-          management_fee:
-            page_header: ""
-            has_management_fee:
-              check_answer_label: "Monthly estate management fee"
-              check_answer_prompt: "Tell us if the property has an estate management fee"
-              hint_text: "Estate management fees are typically used for the maintenance of communal gardens, payments, private roads, car parks and/or play areas within new build estates."
-              question_text: "Does the property have an estate management fee?"
-            management_fee:
-              check_answer_label:  "Monthly estate management fee"
-              check_answer_prompt: ""
-              hint_text: ""
-              question_text: "Enter the total monthly management fee"