Browse Source

Update success banner for deleting duplicates

pull/1752/head
Kat 3 years ago
parent
commit
c112e9fe63
  1. 6
      app/controllers/delete_logs_controller.rb
  2. 3
      config/locales/en.yml
  3. 2
      spec/features/lettings_log_spec.rb

6
app/controllers/delete_logs_controller.rb

@ -26,9 +26,13 @@ class DeleteLogsController < ApplicationController
def discard_lettings_logs
logs = LettingsLog.find(params.require(:ids))
discard logs
if request.referer&.include?("delete-duplicates")
log_ids = logs.map { |log| "Log #{log.id}" }.join(", ")
redirect_to lettings_logs_path, notice: I18n.t("notification.duplicate_logs_deleted", count: logs.count, log_ids:)
else
redirect_to lettings_logs_path, notice: I18n.t("notification.logs_deleted", count: logs.count)
end
end
def delete_sales_logs
@delete_logs_form = delete_logs_form(log_type: :sales)

3
config/locales/en.yml

@ -182,6 +182,9 @@ en:
logs_deleted:
one: "%{count} log has been deleted"
other: "%{count} logs have been deleted"
duplicate_logs_deleted:
one: "%{log_ids} has been deleted"
other: "%{log_ids} have been deleted"
validations:
organisation:

2
spec/features/lettings_log_spec.rb

@ -404,6 +404,8 @@ RSpec.describe "Lettings Log Features" do
click_button "Delete this log"
duplicate_log.reload
expect(duplicate_log.status).to eq("deleted")
expect(page).to have_css(".govuk-notification-banner.govuk-notification-banner--success")
expect(page).to have_content("Log #{duplicate_log.id} has been deleted")
end
end
end

Loading…
Cancel
Save