diff --git a/app/view_objects/teacher_interface/further_information_request_view_object.rb b/app/view_objects/teacher_interface/further_information_request_view_object.rb index 3345eaf68d..2708f1be76 100644 --- a/app/view_objects/teacher_interface/further_information_request_view_object.rb +++ b/app/view_objects/teacher_interface/further_information_request_view_object.rb @@ -88,7 +88,7 @@ def item_name(item) "teacher_interface.further_information_request.show.failure_reason.#{item.failure_reason_key}", ) when "work_history_contact" - "Add your work history contact’s details" + "Add work history details" when "document" "Upload your #{I18n.t("document.document_type.#{item.document.document_type}")} document" end diff --git a/app/views/assessor_interface/further_information_requests/preview.html.erb b/app/views/assessor_interface/further_information_requests/preview.html.erb index 9e8302c41c..c12fbf359f 100644 --- a/app/views/assessor_interface/further_information_requests/preview.html.erb +++ b/app/views/assessor_interface/further_information_requests/preview.html.erb @@ -1,29 +1,43 @@ <% content_for :page_title, "Check the further information you’re asking the applicant for" %> <% content_for :back_link_url, edit_assessor_interface_application_form_assessment_path(@application_form, @assessment) %> -

- Check the further information you’re asking the applicant for -

+

Check the information you’re asking the applicant for

-

This screen shows your reasons for requesting further information, along with your notes to the applicant.

-

If you need to change your notes, you’ll need to go back to the relevant section.

+<% @further_information_request.items.group_by(&:failure_reason_key).each do |failure_reason_key, items| %> + <% if failure_reason_key == "school_details_cannot_be_verified" %> +
+

Reason for request

+ <% if items.size == 1 %> +

We were unable to verify the details for <%= items.first.work_history.school_name %>.

+ <% else %> +

We were unable to verify the details for:

+ + <% end %> +

Note to applicant

-

Your further information request reasons and notes

+ <%= govuk_inset_text do %> + <%= simple_format items.first.failure_reason_assessor_feedback %> + <% end %> +
+ <% else %> + <% items.each do |item| %> +
+

<%= t(item.failure_reason_key, scope: %i[assessor_interface assessment_sections failure_reasons as_statement]) %>

-<% @further_information_request.items.each do |item| %> -
-

<%= t(item.failure_reason_key, scope: %i[assessor_interface assessment_sections failure_reasons as_statement]) %>

-

Your note to the applicant:

+

Note to applicant

- <%= govuk_inset_text do %> - <%= simple_format item.failure_reason_assessor_feedback %> + <%= govuk_inset_text do %> + <%= simple_format item.failure_reason_assessor_feedback %> + <% end %> +
<% end %> -
+ <% end %> <% end %> -

Send the further information request email

-

On the next screen, you’ll see a preview of the email.

-
<%= govuk_button_link_to "Continue", new_assessor_interface_application_form_assessment_further_information_request_path(@application_form, @assessment) %> <%= govuk_button_link_to "Back to overview", assessor_interface_application_form_path(@application_form), secondary: true %> diff --git a/app/views/teacher_interface/application_forms/show/_further_information_requested.html.erb b/app/views/teacher_interface/application_forms/show/_further_information_requested.html.erb index 0a309c147b..48d76076bb 100644 --- a/app/views/teacher_interface/application_forms/show/_further_information_requested.html.erb +++ b/app/views/teacher_interface/application_forms/show/_further_information_requested.html.erb @@ -1,11 +1,5 @@

We need some more information

-

The assessor needs you to provide some additional information so they can continue reviewing your application.

- -
- You may need to upload 1 or more documents. Make sure any files you upload clearly show the whole document or page, and that any text is easy to read. -
- -

You must add all of the requested information, so the assessor can continue reviewing your QTS application.

+

The assessor needs you to provide more information so they can continue reviewing your application.

<%= govuk_start_button(text: "Start now", href: teacher_interface_application_form_further_information_request_path(view_object.further_information_request)) %> diff --git a/config/locales/assessor_interface.en.yml b/config/locales/assessor_interface.en.yml index 5635c46e7d..60924ebd50 100644 --- a/config/locales/assessor_interface.en.yml +++ b/config/locales/assessor_interface.en.yml @@ -198,7 +198,7 @@ en: registration_number: We could not find the applicant’s reference number, or the number was in the wrong format. The applicant will need to supply the number again. registration_number_alternative: We could not find the applicant’s reference number. The applicant will need to upload written proof of recognition as a teacher instead. satisfactory_evidence_work_history: The information provided on work history is not sufficient to award QTS. - school_details_cannot_be_verified: We could not verify the details of one of the schools that the applicant has entered. + school_details_cannot_be_verified: We could not verify school details entered by the applicant. teaching_certificate_illegible: The teaching qualification certificate (or translation) is illegible or in a format that we cannot accept. teaching_hours_not_fulfilled: The required teaching hours have not been fulfilled. teaching_qualification: We were not provided with sufficient evidence to confirm the teaching qualification entered on the online application form. diff --git a/config/locales/helpers.en.yml b/config/locales/helpers.en.yml index 36f91f592f..e49f16639f 100644 --- a/config/locales/helpers.en.yml +++ b/config/locales/helpers.en.yml @@ -8,7 +8,7 @@ en: assessor_interface_assessment_section_form: selected_failure_reasons: Select all options that are relevant to you. failure_reason_notes: - text: This FI reason gives the applicant a free text box. Use this space to give them clear instructions on what they need to provide. + text: This FI allows the applicant to send us a typed response. Use this space to give them clear instructions on what they need to provide. document: This FI reason asks the applicant to upload a file. Use this space to give them clear instructions on what they need to provide. decline: As this reason triggers a decline, you do not need to add a note, but you can use the space below to add anything you feel might be helpful, if you want to. assessor_interface_create_note_form: @@ -278,7 +278,7 @@ en: assessor_interface_assessment_section_form: selected_failure_reasons: What are the reasons for your recommendation? scotland_full_registration: Does the applicant have or are they eligible for full registration? - school_details_cannot_be_verified_work_history_checked: Choose the schools that need reference details to be amended + school_details_cannot_be_verified_work_history_checked: Select the school that we were unable to verify (there may be more than one) assessor_interface_filter_form: action_required_by: Action required by assessor_ids: Assessor name diff --git a/spec/system/assessor_interface/requesting_further_information_spec.rb b/spec/system/assessor_interface/requesting_further_information_spec.rb index 03bde869da..92a8c1dcae 100644 --- a/spec/system/assessor_interface/requesting_further_information_spec.rb +++ b/spec/system/assessor_interface/requesting_further_information_spec.rb @@ -75,9 +75,7 @@ def when_i_select_request_further_information def and_i_see_the_further_information_request_items expect(assessor_request_further_information_page.items.count).to eq(1) - expect( - assessor_request_further_information_page.items.first.heading.text, - ).to eq( + expect(assessor_request_further_information_page).to have_content( "Subjects entered are acceptable for QTS, but the uploaded qualifications do not match them.", ) expect( diff --git a/spec/system/teacher_interface/further_information_spec.rb b/spec/system/teacher_interface/further_information_spec.rb index c34a9a0293..4292ab2ad4 100644 --- a/spec/system/teacher_interface/further_information_spec.rb +++ b/spec/system/teacher_interface/further_information_spec.rb @@ -159,9 +159,7 @@ def and_i_see_the_check_your_answers_items expect(rows.first.key.text).to eq( "Tell us more about the subjects you can teach", ) - expect(rows.second.key.text).to eq( - "Add your work history contact’s details", - ) + expect(rows.second.key.text).to eq("Add work history details") expect(rows.last.key.text).to eq("Upload your identity document") end @@ -233,7 +231,7 @@ def text_check_answers_item def work_history_task_list_item teacher_further_information_requested_page.task_list.find_item( - "Add your work history contact’s details", + "Add work history details", ) end