From e4bf0c3e7d5ae262eb1c8d21a99818a81944d31e Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Fri, 2 Jun 2017 17:01:26 +0100
Subject: [PATCH] =?UTF-8?q?Don=E2=80=99t=20link=20to=20one-off=20reports?=
 =?UTF-8?q?=20from=20activity?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

There’s no need to see the ‘Report’ page for a one-off message – pretty
much all the info about the notification is there on the page. And
‘Sent from _Report_’ looks weird.

We should display it more like an API call instead.
---
 app/templates/views/activity/notifications.html | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/templates/views/activity/notifications.html b/app/templates/views/activity/notifications.html
index 42c6cf169..06158fb75 100644
--- a/app/templates/views/activity/notifications.html
+++ b/app/templates/views/activity/notifications.html
@@ -21,7 +21,10 @@
           {{ item.to }}
         </p>
         <p class="hint">
-          {% if item.job %}
+          {% if item.job and item.job.original_file_name == 'Report' %}
+            <a href="{{ url_for('.view_template_version', service_id=current_service.id, template_id=item.template.id, version=item.template_version) }}">{{ item.template.name }}</a>
+            sent to one recipient
+          {% elif item.job %}
             From <a href="{{ url_for(".view_job", service_id=current_service.id, job_id=item.job.id) }}">{{ item.job.original_file_name }}</a>
           {% else %}
             <a href="{{ url_for('.view_template_version', service_id=current_service.id, template_id=item.template.id, version=item.template_version) }}">{{ item.template.name }}</a>