Fixed Issue emails

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

@@ -1 +1,21 @@
-# TODO: {{ changer }} {{ object }}
+{% extends "emails/base.jinja" %}
+
+{% set final_url = resolve_front_url("issue", object.project.id, object.id) %}
+{% set final_url_name = "Greenmine - View issue #{0}".format(object.ref) %}
+
+{% block body %}
+<table border="0" width="100%" cellpadding="0" cellspacing="0" class="table-body">
+    <tr>
+        <td>
+            <h1>Project: {{ object.project.name }}</h1>
+            <h2>Issue #{{ object.ref }}: {{ object.subject }}</h2>
+            <p>Created by <b>{{ changer.get_full_name() }}</b>.</p>
+        </td>
+    </tr>
+</table>
+{% endblock %}
+{% block footer %}
+<p style="padding: 10px; border-top: 1px solid #eee;">
+    More info at: <a href="{{ final_url }}" style="color: #666;">{{ final_url_name }}</a>
+</p>
+{% endblock %}

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

@@ -1,2 +1,8 @@
-# TODO: {{ changer }} {{ object }}
+{% set final_url = resolve_front_url("issue", object.project.id, object.id) %}
+{% set final_url_name = "Greenmine - View issue #{0}".format(object.ref) %}
 
+- Project: {{ object.project.name }}
+- US #{{ object.ref }}: {{ object.subject }}
+- Created by {{ changer.get_full_name() }}
+
+** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/issues/templates/emails/create_issue_notification-subject.jinja

@@ -1 +1 @@
-Create the new Issue "{{ object }}"
+[{{ object.project.name }}] Created the issue #{{ object.ref }} "{{ object.subject }}"

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

@@ -1,2 +1,13 @@
-# TODO: {{ changer }} {{ object }}
+{% extends "emails/base.jinja" %}
 
+{% block body %}
+<table border="0" width="100%" cellpadding="0" cellspacing="0" class="table-body">
+    <tr>
+        <td>
+            <h1>{{ object.project.name }}</h1>
+            <h2>Issue #{{ object.ref }}: {{ object.subject }}</h2>
+            <p>Deleted by <b>{{ changer.get_full_name() }}</b></p>
+        </td>
+    </tr>
+</table>
+{% endblock %}

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

@@ -1,2 +1,3 @@
-# TODO: {{ changer }} {{ object }}
+- Project: {{ object.project.name }}
-
+- Issue #{{ object.ref }}: {{ object.subject }}
+- Deleted by {{ changer.get_full_name() }}

greenmine/projects/issues/templates/emails/destroy_issue_notification-subject.jinja

@@ -1 +1 @@
-Deleted the issue "{{ object }}"
+[{{ object.project.name }}] Deleted the issue #{{ object.ref }} "{{ object.subject }}"

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

@@ -1,7 +1,40 @@
-<p>Updated fields by {{ changer }}:</p>
+{% extends "emails/base.jinja" %}
+
+{% set final_url = resolve_front_url("issue", object.project.id, object.id) %}
+{% set final_url_name = "Greenmine - View issue #{0}".format(object.ref) %}
+
+{% block body %}
+<table border="0" width="100%" cellpadding="0" cellspacing="0" class="table-body">
+    <tr>
+        <td>
+            <h1>Project: {{ object.project.name }}</h1>
+            <h2>Issue #{{ object.ref }}: {{ object.subject }}</h2>
+            <p>Updated by <b>{{ changer.get_full_name() }}</b>.</p>
+            <p>Updated fields:
+            {% for field in changed_fields_dict.values() %}
+                <b>{{ field.verbose_name }}</b>{% if not loop.last %},{% endif %}
+            {% endfor %}
+            </p>
+
+        {# TODO: Improvemments in some methods to use that:  #}
+        {#         - WatchedMixin._get_changed_field_old_value() #}
+        {#         - WatchedMixin._get_changed_field_new_value() #}
+        {#
             <ul>
             {% for field in changed_fields_dict.values() %}
-    <li><b>{{ field.verbose_name}}</b>: from "{{ field.old_value}}" to "{{ field.new_value}}".</li>
+                <li>
+                    <b>{{ field.verbose_name}}</b>: from <i>{{ field.old_value}}</i>
+                                                      to <i>{{ field.new_value}}</i>.
+                </li>
             {% endfor %}
             </ul>
-{# TODO Print the "object" details #}
+        #}
+        </td>
+    </tr>
+</table>
+{% endblock %}
+{% block footer %}
+<p style="padding: 10px; border-top: 1px solid #eee;">
+    More info at: <a href="{{ final_url }}" style="color: #666;">{{ final_url_name }}</a>
+</p>
+{% endblock %}

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

@@ -1,9 +1,21 @@
-Updated fields by {{ changer }}:
+{% set final_url = resolve_front_url("issue", object.project.id, object.id) %}
+{% set final_url_name = "Greenmine - View issue #{0}".format(object.ref) %}
 
+- Project: {{ object.project.name }}
+- Issue #{{ object.ref }}: {{ object.subject }}
+- Updated by {{ changer.get_full_name() }}
+- Updated fields:
 {% for field in changed_fields_dict.values() %}
-    ** {{ field.verbose_name}}: from "{{ field.old_value}}" to "{{ field.new_value}}".
+    {{ field.verbose_name }}{% if not loop.last %}, {% endif %}
 {% endfor %}
 
-{# TODO Print the "object" details #}
+{# TODO: Improvemments in some methods to use that:  #}
-
+{#         - WatchedMixin._get_changed_field_old_value() #}
+{#         - WatchedMixin._get_changed_field_new_value() #}
+{#
+{% for field in changed_fields_dict.values() %}
+    * {{ field.verbose_name}}</b>: from '{{ field.old_value}}' to '{{ field.new_value }}'.
+{% endfor %}
+#}
 
+** More info at {{ final_url_name }} ({{ final_url }}) **

greenmine/projects/issues/templates/emails/update_issue_notification-subject.jinja

@@ -1 +1 @@
-Updated the issue "{{ object }}"
+[{{ object.project.name }}] Updated the issue #{{ object.ref }} "{{ object.subject }}"

greenmine/projects/issues/templates/emails/update_user_story_notification-subject.jinja

@@ -1 +0,0 @@
-Updated the user story "{{ object }}"