[change] Configured target_link for config error notification

openwisp_controller/config/apps.py

@@ -176,6 +176,10 @@ def register_notification_types(self):
                     'The last working configuration has been restored from a backup '
                     'present on the filesystem of the device.'
                 ),
+                'target_link': (
+                    'openwisp_controller.config.utils'
+                    '.get_config_error_notification_target_url'
+                ),
             },
             models=[self.device_model, self.config_model],
         )

openwisp_controller/config/tests/test_notifications.py

@@ -37,6 +37,7 @@ def test_config_problem_notification(self):
             ' configuration encountered an error',
         )
         self.assertIn('encountered an error', notification.message)
+        self.assertEqual(notification.target_url.endswith('#config-group'), True)
 
     def test_device_registered(self):
         # To avoid adding repetitive code for registering a device,

openwisp_controller/config/utils.py

@@ -5,6 +5,7 @@
 from django.http import Http404, HttpResponse
 from django.shortcuts import get_object_or_404 as base_get_object_or_404
 from django.urls import path, re_path
+from openwisp_notifications.utils import _get_object_link
 
 logger = logging.getLogger(__name__)
 
@@ -199,3 +200,8 @@ def get_default_templates_queryset(
     if backend:
         queryset = queryset.filter(backend=backend)
     return queryset
+
+
+def get_config_error_notification_target_url(obj, field, absolute_url=True):
+    url = _get_object_link(obj, field, absolute_url)
+    return f'{url}#config-group'