feature: set enabled=True for initial grouped errors visit

Author: chrisdoehring

cdip_admin/integrations/templates/integrations/bridge_integration_list.html

@@ -22,7 +22,7 @@ <h2 class="align-bottom">Bridge Integrations</h2>
 
     {% if error_count > 0 %}
     <div class="alert alert-warning mt-2">
-        <strong>{{ error_count }}</strong> of {{ total_count }} configurations have errors
+        <strong>{{ error_count }}</strong> of {{ total_count }} enabled configurations have errors
     </div>
     {% endif %}
 

cdip_admin/integrations/templates/integrations/inbound_integration_configuration_list.html

@@ -21,7 +21,7 @@ <h2>Inbound Integrations</h2>
 
     {% if error_count > 0 %}
     <div class="alert alert-warning mt-2 d-flex justify-content-between align-items-center">
-        <span><strong>{{ error_count }}</strong> of {{ total_count }} configurations have errors</span>
+        <span><strong>{{ error_count }}</strong> of {{ total_count }} enabled configurations have errors</span>
         <a href="{% url 'inbound_integration_errors' %}{% if request.GET.urlencode %}?{{ request.GET.urlencode }}{% endif %}" class="btn btn-sm btn-warning">View grouped errors</a>
     </div>
     {% endif %}

cdip_admin/integrations/templates/integrations/inbound_integration_errors.html

@@ -108,6 +108,19 @@ <h2 class="mb-0 mr-3">Inbound Integration Errors</h2>
             });
             if (!el.value) ts.clear();
         });
+
+        var hlParams = new URLSearchParams(location.search);
+        if (hlParams.get('_hl') === 'enabled') {
+            hlParams.delete('_hl');
+            history.replaceState(null, '', location.pathname + '?' + hlParams.toString());
+            var enabledEl = document.querySelector('#inbound-filter-form select[name="enabled"]');
+            var wrapper = enabledEl && enabledEl.parentElement;
+            if (wrapper) {
+                wrapper.style.transition = 'box-shadow 1s';
+                wrapper.style.boxShadow = '0 0 0 3px #28a745';
+                setTimeout(function() { wrapper.style.boxShadow = ''; }, 1200);
+            }
+        }
     })();
 </script>
 {% endif %}

cdip_admin/integrations/templates/integrations/outbound_integration_configuration_list.html

@@ -23,7 +23,7 @@ <h2>Outbound Integrations</h2>
 
     {% if error_count > 0 %}
     <div class="alert alert-warning mt-2">
-        <strong>{{ error_count }}</strong> of {{ total_count }} configurations have errors
+        <strong>{{ error_count }}</strong> of {{ total_count }} enabled configurations have errors
     </div>
     {% endif %}
 

cdip_admin/integrations/views.py

@@ -1196,8 +1196,8 @@ def get_context_data(self, **kwargs):
         base_url = reverse("inbound_integration_configuration_list")
         context["base_url"] = base_url
         qs = self.filterset.qs
-        context["error_count"] = qs.filter(state__has_key='error').count()
-        context["total_count"] = qs.count()
+        context["error_count"] = qs.filter(state__has_key='error', enabled=True).count()
+        context["total_count"] = qs.filter(enabled=True).count()
         return context
 
     def get_queryset(self):
@@ -1218,6 +1218,14 @@ def get_queryset(self):
 class InboundIntegrationErrorsView(LoginRequiredMixin, TemplateView):
     template_name = "integrations/inbound_integration_errors.html"
 
+    def get(self, request, *args, **kwargs):
+        if 'enabled' not in request.GET:
+            params = request.GET.copy()
+            params['enabled'] = 'true'
+            params['_hl'] = 'enabled'
+            return redirect(request.path + '?' + params.urlencode())
+        return super().get(request, *args, **kwargs)
+
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         qs = InboundIntegrationConfiguration.objects.get_queryset()
@@ -1533,8 +1541,8 @@ def get_context_data(self, **kwargs):
         base_url = reverse("outbound_integration_configuration_list")
         context["base_url"] = base_url
         qs = self.filterset.qs
-        context["error_count"] = qs.filter(state__has_key='error').count()
-        context["total_count"] = qs.count()
+        context["error_count"] = qs.filter(state__has_key='error', enabled=True).count()
+        context["total_count"] = qs.filter(enabled=True).count()
         return context
 
     def get_queryset(self):
@@ -1573,8 +1581,8 @@ def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         context["base_url"] = reverse("bridge_integration_list")
         qs = self.filterset.qs
-        context["error_count"] = qs.filter(state__has_key='error').count()
-        context["total_count"] = qs.count()
+        context["error_count"] = qs.filter(state__has_key='error', enabled=True).count()
+        context["total_count"] = qs.filter(enabled=True).count()
         return context
 
     def get_queryset(self):