Split report and reminder (nag) templates

hc/accounts/models.py

@@ -239,10 +239,10 @@ class Profile(models.Model):
             for check in checks:
                 check.past_downtimes = check.downtimes_by_boundary(boundaries)[:-1]
 
-            ctx["nag"] = False
             ctx["checks"] = checks
             ctx["boundaries"] = boundaries[:-1]
             ctx["monthly_or_weekly"] = self.reports
+            emails.report(self.user.email, ctx, headers)
 
         if nag:
             # For nags, only show checks that are currently down
@@ -251,10 +251,9 @@ class Profile(models.Model):
                 return False
             ctx["checks"] = checks
             ctx["num_down"] = len(checks)
-            ctx["nag"] = True
             ctx["nag_period"] = self.nag_period.total_seconds()
+            emails.nag(self.user.email, ctx, headers)
 
-        emails.report(self.user.email, ctx, headers)
         return True
 
     def sms_sent_this_month(self):

hc/api/tests/test_sendreports.py

@@ -21,7 +21,6 @@ MOCK_NOW = Mock(return_value=CURRENT_TIME)
 NAG_TEXT = """Hello,
 
 This is a hourly reminder sent by Mychecks.
-
 One check is currently DOWN:
 
 

hc/lib/emails.py

@@ -103,6 +103,11 @@ def report(to, ctx, headers={}):
     send(m, block=True)
 
 
+def nag(to, ctx, headers={}):
+    m = make_message("nag", to, ctx, headers=headers)
+    send(m, block=True)
+
+
 def deletion_notice(to, ctx, headers={}):
     m = make_message("deletion-notice", to, ctx, headers=headers)
     send(m, block=True)

templates/emails/nag-body-html.html

@@ -0,0 +1,34 @@
+{% extends "emails/base.html" %}
+{% load humanize hc_extras %}
+
+{% block content %}
+Hello,<br />
+
+This is a
+{% if nag_period == 3600 %}hourly{% endif %}
+{% if nag_period == 86400 %}daily{% endif %}
+reminder sent by <a href="{% site_root %}">{% site_name %}</a>.<br />
+
+{% if num_down == 1%}
+One check is currently <strong>DOWN</strong>:<br />
+{% else %}
+{{ num_down }} checks are currently <strong>DOWN</strong>:<br />
+{% endif %}
+
+{% include "emails/nag-summary-html.html" %}
+
+<strong>Too many notifications?</strong>
+Visit the <a href="{{ notifications_url }}">Email Reports</a>
+page on {% site_name %} to set your notification preferences.
+<br /><br />
+
+Cheers,<br>
+The {% site_name %} Team
+{% endblock %}
+
+{% block unsub %}
+<br>
+<a href="{{ unsub_link }}" target="_blank" style="color: #666666; text-decoration: underline;">
+    Unsubscribe
+</a>
+{% endblock %}

templates/emails/nag-body-text.html

@@ -0,0 +1,10 @@
+{% load hc_extras %}Hello,
+
+This is a {% if nag_period == 3600 %}hourly {% endif %}{% if nag_period == 86400 %}daily {% endif %}reminder sent by {% site_name %}.
+{% if num_down == 1%}One check is currently DOWN:{% else %}{{ num_down }} checks are currently DOWN:{% endif %}
+{% include 'emails/summary-text.html' %}
+
+
+--
+Cheers,
+{% site_name %}

templates/emails/nag-subject.html

@@ -0,0 +1 @@
+Reminder: {{ num_down }} check{{ num_down|pluralize }} still down

templates/emails/report-body-html.html

@@ -4,38 +4,14 @@
 {% block content %}
 Hello,<br />
 
-{% if nag %}
+This is a {{ monthly_or_weekly }} report sent by
-    This is a
+<a href="{% site_root %}">{% site_name %}</a>.<br />
-    {% if nag_period == 3600 %}hourly{% endif %}
-    {% if nag_period == 86400 %}daily{% endif %}
-    reminder sent by <a href="{% site_root %}">{% site_name %}</a>.<br />
 
-    {% if num_down == 1%}
+{% include "emails/report-summary-html.html" %}
-        One check is currently <strong>DOWN</strong>:
-    {% else %}
-        {{ num_down }} checks are currently <strong>DOWN</strong>:
-    {% endif %}
-
-    <br />
-    {% include "emails/summary-html.html" %}
-
-    <strong>Too many notifications?</strong>
-    Visit the <a href="{{ notifications_url }}">Email Reports</a>
-    page on {% site_name %} to set your notification preferences.
-
-{% else %}
-    This is a {{ monthly_or_weekly }} report
-    sent by <a href="{% site_root %}">{% site_name %}</a>.
-
-    <br />
-    {% include "emails/summary-downtimes-html.html" %}
-
-    <strong>Just one more thing to check:</strong>
-    Do you have more cron jobs,
-    not yet on this list, that would benefit from monitoring?
-    Get the ball rolling by adding one more!
-{% endif %}
 
+<strong>Just one more thing to check:</strong>
+Do you have more cron jobs, not yet on this list, that would benefit from monitoring?
+Get the ball rolling by adding one more!
 <br /><br />
 
 Cheers,<br>

templates/emails/report-body-text.html

@@ -1,8 +1,6 @@
 {% load hc_extras %}Hello,
 
-{% if nag %}This is a {% if nag_period == 3600 %}hourly {% endif %}{% if nag_period == 86400 %}daily {% endif %}reminder sent by {% site_name %}.
+This is a {{ monthly_or_weekly }} report sent by {% site_name %}.
-
-{% if num_down == 1%}One check is currently DOWN:{% else %}{{ num_down }} checks are currently DOWN:{% endif %}{% else %}This is a {{ monthly_or_weekly }} report sent by {% site_name %}.{% endif %}
 {% include 'emails/summary-text.html' %}
 
 

templates/emails/report-subject.html

@@ -1,6 +1 @@
-{% if nag %}
+{{ monthly_or_weekly|capfirst }} Report
-    Reminder: {{ num_down }} check{{ num_down|pluralize }} still down
-{% else %}
-    {{ monthly_or_weekly|capfirst }} Report
-{% endif %}
-