diff --git a/app/views/logs/download_csv.html.erb b/app/views/logs/download_csv.html.erb index edcf25543..3ef435616 100644 --- a/app/views/logs/download_csv.html.erb +++ b/app/views/logs/download_csv.html.erb @@ -11,6 +11,34 @@

We'll send a secure download link to your email address <%= @current_user.email %>.

You've selected <%= count %> logs.

+
+
+

+ Check your filters +

+ + <%= govuk_summary_list do |summary_list| %> + <% check_your_answers_lettings_filters_list(session_filters).each do |filter| %> + <% summary_list.with_row do |row| %> + <% row.with_key { filter[:label] } %> + <% row.with_value do %> + <%= simple_format( + filter[:value], + wrapper_tag: "span", + class: "govuk-!-margin-right-4", + ) %> + <% end %> + + <% row.with_action( + text: "Change", + href: "#{filter[:path]}?search=#{search_term}&codes_only=#{codes_only}", + ) %> + <% end %> + <% end %> + <% end %> +
+
+ <%= govuk_button_to "Send email", post_path, method: :post, params: { search: search_term, codes_only: } %> diff --git a/spec/requests/lettings_logs_controller_spec.rb b/spec/requests/lettings_logs_controller_spec.rb index 9d49fb7f4..3daebe8fd 100644 --- a/spec/requests/lettings_logs_controller_spec.rb +++ b/spec/requests/lettings_logs_controller_spec.rb @@ -1421,6 +1421,17 @@ RSpec.describe LettingsLogsController, type: :request do it "includes the search term" do expect(page).to have_field("search", type: "hidden", with: search_term) end + + it "allows updating log filters" do + expect(page).to have_content("Check your filters") + expect(page).to have_link("Change", count: 6) + expect(page).to have_link("Change", href: "/lettings-logs/filters/years?search=#{search_term}&codes_only=false") + expect(page).to have_link("Change", href: "/lettings-logs/filters/assigned-to?search=#{search_term}&codes_only=false") + expect(page).to have_link("Change", href: "/lettings-logs/filters/owned-by?search=#{search_term}&codes_only=false") + expect(page).to have_link("Change", href: "/lettings-logs/filters/managed-by?search=#{search_term}&codes_only=false") + expect(page).to have_link("Change", href: "/lettings-logs/filters/status?search=#{search_term}&codes_only=false") + expect(page).to have_link("Change", href: "/lettings-logs/filters/needstype?search=#{search_term}&codes_only=false") + end end context "when there are no years selected in the filters" do