Merge pull request #7565 from readthedocs/humitos/context-procesor-no…

…tifications Add our `readthedocs_processor` data to our notifications
readthedocs · Oct 22, 2020 · c85171a · c85171a
2 parents 50f0fd0 + 656862a
commit c85171a
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 1 deletion.
diff --git a/readthedocs/notifications/notification.py b/readthedocs/notifications/notification.py
@@ -3,6 +3,7 @@
 """Support for templating of notifications."""
 
 import logging
+from readthedocs.core.context_processors import readthedocs_processor
 
 from django.conf import settings
 from django.db import models
@@ -50,14 +51,16 @@ def get_subject(self):
         return template.render(context=Context(self.get_context_data()))
 
     def get_context_data(self):
-        return {
+        context = {
             self.context_object_name: self.object,
             'request': self.request,
             'production_uri': '{scheme}://{host}'.format(
                 scheme='https',
                 host=settings.PRODUCTION_DOMAIN,
             ),
         }
+        context.update(readthedocs_processor(self.request))
+        return context
 
     def get_template_names(self, backend_name, source_format=constants.HTML):
         names = []

diff --git a/readthedocs/rtd_tests/tests/test_notifications.py b/readthedocs/rtd_tests/tests/test_notifications.py
@@ -67,6 +67,18 @@ class TestNotification(Notification):
                 'foo': build,
                 'production_uri': 'https://readthedocs.org',
                 'request': req,
+
+                # readthedocs_processor context
+                'DASHBOARD_ANALYTICS_CODE': None,
+                'DO_NOT_TRACK_ENABLED': False,
+                'GLOBAL_ANALYTICS_CODE': None,
+                'PRODUCTION_DOMAIN': 'readthedocs.org',
+                'PUBLIC_DOMAIN': None,
+                'SITE_ROOT': mock.ANY,
+                'SUPPORT_EMAIL': None,
+                'TEMPLATE_ROOT': mock.ANY,
+                'USE_PROMOS': False,
+                'USE_SUBDOMAIN': False,
             },
         )
 
@@ -220,6 +232,18 @@ def test_context_data(self):
             'request': None,
             'production_uri': 'https://readthedocs.org',
             'other': {'name': 'other name'},
+
+            # readthedocs_processor context
+            'DASHBOARD_ANALYTICS_CODE': None,
+            'DO_NOT_TRACK_ENABLED': False,
+            'GLOBAL_ANALYTICS_CODE': None,
+            'PRODUCTION_DOMAIN': 'readthedocs.org',
+            'PUBLIC_DOMAIN': None,
+            'SITE_ROOT': mock.ANY,
+            'SUPPORT_EMAIL': None,
+            'TEMPLATE_ROOT': mock.ANY,
+            'USE_PROMOS': False,
+            'USE_SUBDOMAIN': False,
         }
         self.assertEqual(self.n.get_context_data(), context)