Merge pull request #2959 from kobotoolbox/2750-Point-URL-in-REST-submission-email-notification-to-submission-list

Fix to point the URL in the REST submission email to the correct screen in the UI

Author: noliveleger

kobo/apps/hook/tasks.py

@@ -110,6 +110,7 @@ def failures_reports():
                 max_length = 0
                 records[record.hook.asset.owner.id]['assets'][record.hook.asset.uid] = {
                     'name': record.hook.asset.name,
+                    'hook_uid': record.hook.uid,
                     'max_length': 0,
                     'logs': []
                 }

kobo/apps/hook/templates/reports/failures_email_body.html

@@ -12,7 +12,7 @@
 
 {% for asset_uid, asset in assets.items %}
   {% url 'api_v2:asset-detail' uid=asset_uid as asset_url %}
-  <p>{{ i18n_asset }}: <b><a href="{{ kpi_base_url }}{{ asset_url }}">{{ asset.name }}</a></b></p>
+  <p>{{ i18n_asset }}: <b><a href="{{ kpi_base_url }}/#/forms/{{asset_uid}}/settings/rest/{{asset.hook_uid}}">{{ asset.name }}</a></b></p>
     <table>
         <tr>
           <th style="border-bottom:1px solid black;">{{ i18n_hook }}</th>

kobo/apps/hook/templates/reports/failures_email_body.txt

@@ -13,7 +13,7 @@
 
 {% for asset_uid, asset in assets.items %}
 {% url 'api_v2:asset-detail' uid=asset_uid as asset_url %}
-{{ i18n_asset }}: {{ asset.name }} - [{{ kpi_base_url }}{{ asset_url }}]
+{{ i18n_asset }}: {{ asset.name }} - [{{ kpi_base_url }}/#/forms/{{asset_uid}}/settings/rest/{{asset.hook_uid}}]
 
     {% with max_length=asset.max_length|add:"2" %}
     {{ i18n_hook|center:max_length }}|{{ i18n_submission|center:25 }}|{{ i18n_status_code|center:15 }}|{{ i18n_message|truncatechars:23|center:25 }}|{{ i18n_date|center:25 }}

kobo/apps/hook/tests/test_email.py

@@ -36,6 +36,7 @@ def test_notifications(self):
             "assets": {
                 self.asset.uid: {
                     "name": self.asset.name,
+                    "hook_uid": self.hook.uid,
                     "max_length": len(self.hook.name),
                     "logs": [{
                         "hook_name": self.hook.name,