[feature] Batch email notifications

openwisp_notifications/handlers.py

@@ -25,7 +25,6 @@
 logger = logging.getLogger(__name__)
 
 EXTRA_DATA = app_settings.get_config()['USE_JSONFIELD']
-EMAIL_BATCH_INTERVAL = app_settings.EMAIL_BATCH_INTERVAL
 
 User = get_user_model()
 
@@ -205,6 +204,7 @@ def send_email_notification(sender, instance, created, **kwargs):
             'email_id': instance.recipient.email,
         },
     )
+    EMAIL_BATCH_INTERVAL = app_settings.EMAIL_BATCH_INTERVAL
 
     if cache_data['last_email_sent_time'] and EMAIL_BATCH_INTERVAL > 0:
         # Case 1: Batch email sending logic

openwisp_notifications/tests/test_notifications.py

@@ -11,7 +11,7 @@
 from django.core.exceptions import ImproperlyConfigured
 from django.db.models.signals import post_migrate, post_save
 from django.template import TemplateDoesNotExist
-from django.test import TransactionTestCase, override_settings
+from django.test import TransactionTestCase
 from django.urls import reverse
 from django.utils import timezone
 from django.utils.timesince import timesince
@@ -984,7 +984,7 @@ def test_batch_email_notification_with_call_to_action(self, mock_send_email):
         self.assertIn('View all Notifications', mail.outbox[1].body)
         self.assertNotIn('Test Notification', mail.outbox[1].body)
 
-    @override_settings(EMAIL_BATCH_INTERVAL=0)
+    @patch.object(app_settings, 'EMAIL_BATCH_INTERVAL', 0)
     def test_without_batch_email_notification(self):
         self.notification_options.update(
             {