Display lettings filters CYA

app/views/logs/download_csv.html.erb

@@ -11,6 +11,34 @@
     <p class="govuk-body">We'll send a secure download link to your email address <strong><%= @current_user.email %></strong>.</p>
     <p class="govuk-body">You've selected <%= count %> logs.</p>
 
+    <div class="govuk-grid-row">
+      <div class="govuk-grid-column-three-quarters-from-desktop">
+        <h1 class="govuk-heading-l">
+          Check your filters
+        </h1>
+
+        <%= 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 %>
+      </div>
+    </div>
+
     <%= govuk_button_to "Send email", post_path, method: :post, params: { search: search_term, codes_only: } %>
   </div>
 </div>

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