Merge pull request #1871 from mysociety/1870-add-request-to-excel-ana…

…lyzer-emails [ExcelAnalyzer] Add admin request link to email notifications
mysociety · May 15, 2024 · fe43756 · fe43756
2 parents a925ff7 + 77cdf9a
commit fe43756
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/excel_analyzer.rb b/lib/excel_analyzer.rb
@@ -23,6 +23,7 @@ class ExcelAnalyzerNotifier < ApplicationMailer
 
     def report(foi_attachment, metadata)
       @foi_attachment = foi_attachment
+      @incoming_message = foi_attachment.incoming_message
       @metadata = metadata
 
       from = email_address_with_name(

diff --git a/lib/views/excel_analyzer_notifier/report.text.erb b/lib/views/excel_analyzer_notifier/report.text.erb
@@ -1,7 +1,8 @@
 ExcelAnalyzer has flagged and automatically set hidden prominence of a received
 spreadsheet due to the detection of potentially suspect hidden data.
 
-Admin URL: <%= edit_admin_foi_attachment_url(@foi_attachment) %>
+Admin request URL: <%= admin_request_url(@incoming_message.info_request_id) %>
+Admin attachment URL: <%= edit_admin_foi_attachment_url(@foi_attachment) %>
 
 The following was detected:
 <% @metadata.each do |key, value| %>