Fixed bug #88 (id 94): Added comments to the updated emails

greenmine/base/notifications/api.py

@@ -18,7 +18,8 @@ class NotificationSenderMixin(object):
         super().post_save(obj, created)
 
         users = obj.get_watchers_to_notify(self.request.user)
-        context = {'changer': self.request.user, 'object': obj}
+        comment = self.request.DATA.get("comment", None)
+        context = {'changer': self.request.user, "comment": comment, 'object': obj}
 
         if created:
             self._send_notification_email(self.create_notification_template,

greenmine/projects/issues/templates/emails/update_issue_notification-body-html.jinja

@@ -10,6 +10,10 @@
             <h1>Project: {{ object.project.name }}</h1>
             <h2>Issue #{{ object.ref }}: {{ object.subject }}</h2>
             <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+        {% if comment %}
+            <p>Comment <b>{{ comment|linebreaksbr }}</b></p>
+        {% endif %}
+        {% if changed_fields %}
             <p>Updated fields:
             <dl>
             {% for field in changed_fields %}
@@ -28,6 +32,7 @@
                 {% endif %}
             {% endfor %}
             </dl>
+        {% endif %}
         </td>
     </tr>
 </table>

greenmine/projects/issues/templates/emails/update_issue_notification-body-text.jinja

@@ -4,9 +4,14 @@
 - Project: {{ object.project.name }}
 - Issue #{{ object.ref }}: {{ object.subject }}
 - Updated by {{ changer.get_full_name() }}
+{% if comment %}
+Comment: {{ comment|linebreaksbr }}
+{% endif %}
+{% if changed_fields %}
 - Updated fields:
     {% for field in changed_fields %}
     * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
     {% endfor %}
+{% endif %}
 
 ** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/milestones/templates/emails/update_milestone_notification-body-html.jinja

@@ -10,6 +10,10 @@
             <h1>Project: {{ object.project.name }}</h1>
             <h2>Milestone #{{ object.slug }}: {{ object.name }}</h2>
             <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+        {% if comment %}
+            <p>Comment <b>{{ comment|linebreaksbr }}</b></p>
+        {% endif %}
+        {% if changed_fields %}
             <p>Updated fields:
             <dl>
             {% for field in changed_fields %}
@@ -28,6 +32,7 @@
                 {% endif %}
             {% endfor %}
             </dl>
+        {% endif %}
         </td>
     </tr>
 </table>

greenmine/projects/milestones/templates/emails/update_milestone_notification-body-text.jinja

@@ -4,9 +4,14 @@
 - Project: {{ object.project.name }}
 - Milestone #{{ object.slug }}: {{ object.name }}
 - Updated by {{ changer.get_full_name() }}
+{% if comment %}
+Comment: {{ comment|linebreaksbr }}
+{% endif %}
+{% if changed_fields %}
 - Updated fields:
     {% for field in changed_fields %}
     * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
     {% endfor %}
+{% endif %}
 
 ** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/tasks/templates/emails/update_task_notification-body-html.jinja

@@ -10,6 +10,10 @@
             <h1>Project: {{ object.project.name }}</h1>
             <h2>Task #{{ object.ref }}: {{ object.subject }}</h2>
             <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+        {% if comment %}
+            <p>Comment <b>{{ comment|linebreaksbr }}</b></p>
+        {% endif %}
+        {% if changed_fields %}
             <p>Updated fields:
             <dl>
             {% for field in changed_fields %}
@@ -28,6 +32,7 @@
                 {% endif %}
             {% endfor %}
             </dl>
+        {% endif %}
         </td>
     </tr>
 </table>

greenmine/projects/tasks/templates/emails/update_task_notification-body-text.jinja

@@ -4,9 +4,14 @@
 - Project: {{ object.project.name }}
 - Task #{{ object.ref }}: {{ object.subject }}
 - Updated by {{ changer.get_full_name() }}
+{% if comment %}
+Comment: {{ comment|linebreaksbr }}
+{% endif %}
+{% if changed_fields %}
 - Updated fields:
     {% for field in changed_fields %}
     * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
     {% endfor %}
+{% endif %}
 
 ** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/templates/emails/update_project_notification-body-html.jinja

@@ -9,6 +9,10 @@
         <td>
             <h2>Project #{{ object.slug }}: {{ object.name }}</h2>
             <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+        {% if comment %}
+            <p>Comment <b>{{ comment|linebreaksbr }}</b></p>
+        {% endif %}
+        {% if changed_fields %}
             <p>Updated fields:
             <dl>
             {% for field in changed_fields %}
@@ -27,6 +31,7 @@
                 {% endif %}
             {% endfor %}
             </dl>
+        {% endif %}
         </td>
     </tr>
 </table>

greenmine/projects/templates/emails/update_project_notification-body-text.jinja

@@ -3,9 +3,14 @@
 
 - Project #{{ object.slug }}: {{ object.name }}
 - Updated by {{ changer.get_full_name() }}
+{% if comment %}
+Comment: {{ comment|linebreaksbr }}
+{% endif %}
+{% if changed_fields %}
 - Updated fields:
     {% for field in changed_fields %}
     * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
     {% endfor %}
+{% endif %}
 
 ** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/userstories/templates/emails/update_userstory_notification-body-html.jinja

@@ -10,6 +10,10 @@
             <h1>Project: {{ object.project.name }}</h1>
             <h2>US #{{ object.ref }}: {{ object.subject }}</h2>
             <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+        {% if comment %}
+            <p>Comment <b>{{ comment|linebreaksbr }}</b></p>
+        {% endif %}
+        {% if changed_fields %}
             <p>Updated fields:
             <dl>
             {% for field in changed_fields %}
@@ -28,6 +32,7 @@
                 {% endif %}
             {% endfor %}
             </dl>
+        {% endif %}
         </td>
     </tr>
 </table>

greenmine/projects/userstories/templates/emails/update_userstory_notification-body-text.jinja

@@ -4,9 +4,14 @@
 - Project: {{ object.project.name }}
 - US #{{ object.ref }}: {{ object.subject }}
 - Updated by {{ changer.get_full_name() }}
+{% if comment %}
+Comment: {{ comment|linebreaksbr }}
+{% endif %}
+{% if changed_fields %}
 - Updated fields:
     {% for field in changed_fields %}
     * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
     {% endfor %}
+{% endif %}
 
 ** More info at {{ final_url_name }} ({{ final_url }}) **