Add colon after request summary intro

Apple Mail (and potentially others) renders the next line immediately after this line, so it reads as a continuation which gets confusing: > Here's your daily request summary from WhatDoTheyKnow Production, > and supply of foo… After this change we'll have: > Here's your daily request summary from WhatDoTheyKnow: Production, > and supply of foo…
mysociety · Sep 11, 2023 · 67e56cd · 67e56cd
1 parent 9c2d9b6
commit 67e56cd
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/views/notification_mailer/daily_summary.text.erb b/app/views/notification_mailer/daily_summary.text.erb
@@ -1,4 +1,4 @@
-<%= _("Here's your daily request summary from {{site_name}}",
+<%= _("Here's your daily request summary from {{site_name}}:",
       site_name: site_name.html_safe) %>
 
 <% @grouped_notifications.each do |request, notifications| %>

diff --git a/spec/fixtures/files/notification_mailer/daily-summary.txt b/spec/fixtures/files/notification_mailer/daily-summary.txt
@@ -1,4 +1,4 @@
-Here's your daily request summary from Alaveteli
+Here's your daily request summary from Alaveteli:
 
 The cost of paperclips
 ----------------------

diff --git a/spec/views/notification_mailer/daily_summary.text.erb_spec.rb b/spec/views/notification_mailer/daily_summary.text.erb_spec.rb
@@ -16,6 +16,6 @@
     render
     expect(response).to match("the l'Information team")
     expect(response).
-      to match("Here's your daily request summary from l'Information")
+      to match("Here's your daily request summary from l'Information:")
   end
 end