Skip to content

Commit

Permalink
Merge pull request #3156 from DFE-Digital/LUPEYALPHA-971/rejection-re…
Browse files Browse the repository at this point in the history
…asons

Add rejection reasons for FE claims
  • Loading branch information
rjlynch authored Sep 6, 2024
2 parents 70e2766 + dbe7439 commit c9e1975
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 5 deletions.
14 changes: 9 additions & 5 deletions app/models/policies/further_education_payments.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,15 @@ module FurtherEducationPayments

# Options shown to admins when rejecting a claim
ADMIN_DECISION_REJECTED_REASONS = [
# FIXME RL: this `placeholder` is required to make the
# `spec/models/policies/further_education_payments/claim_personal_data_scrubber_spec.rb`
# test pass. Once we add a real rejection reason we can remove this
# placeholder. Figured this was better than removing the test!
:placeholder
:no_teaching_responsibilities,
:no_eligible_contract_of_employment,
:works_less_than_2_point_5_hours_per_week,
:has_worked_in_further_education_for_more_than_5_years,
:ineligible_subject_or_courses,
:insufficient_time_spent_teaching_eligibble_students,
:duplicate_claim,
:no_response,
:other
]

# TODO: This is needed once the reply-to email address has been added to Gov Notify
Expand Down
11 changes: 11 additions & 0 deletions config/locales/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -836,6 +836,17 @@ en:
url: "https://www.gov.uk/guidance/levelling-up-premium-payments-for-fe-teachers"
fe_provider_list_url: "https://assets.publishing.service.gov.uk/media/66d70f9c701781e1b341dba7/List_of_eligible_FE_providers_and_payment_values_for_targeted_retention_incentives.xlsx"
admin:
decision:
rejected_reasons:
no_teaching_responsibilities: "No teaching responsibilities"
no_eligible_contract_of_employment: "No eligible contract of employment"
works_less_than_2_point_5_hours_per_week: "Works less than 2.5 hours per week (either currently or next term)"
has_worked_in_further_education_for_more_than_5_years: "Has worked in further education for more than 5 years"
ineligible_subject_or_courses: "Ineligible subjects or courses"
insufficient_time_spent_teaching_eligibble_students: "Insufficient time spent teaching eligible students"
duplicate_claim: "Duplicate claim"
no_response: "No response"
other: "Other"
task_questions:
matching_details:
title: Is this claim still valid despite having matching details with other claims?
Expand Down

0 comments on commit c9e1975

Please sign in to comment.