fix: fix accredible models redirect and handle errors on sync

Author: kyrylo-kh

credentials/apps/badges/admin.py

@@ -11,6 +11,7 @@
 from django.utils.safestring import mark_safe
 from django.utils.translation import gettext_lazy as _
 
+from credentials.apps.badges.exceptions import BadgeProviderError
 from credentials.apps.badges.admin_forms import (
     BadgePenaltyForm,
     BadgeRequirementForm,
@@ -38,6 +39,11 @@
 from credentials.apps.badges.toggles import is_badges_enabled
 
 
+ADMIN_CHANGE_VIEW_REVERSE_NAMES = {
+    CredlyBadgeTemplate.ORIGIN: "admin:badges_credlybadgetemplate_change",
+    AccredibleGroup.ORIGIN: "admin:badges_accrediblegroup_change",
+}
+
 class BadgeRequirementInline(admin.TabularInline):
     """
     Badge template requirement inline setup.
@@ -388,14 +394,19 @@ def template_link(self, instance):
         """
         Interactive link to parent (badge template).
         """
-        url = reverse("admin:badges_credlybadgetemplate_change", args=[instance.template.pk])
+        reverse_name = ADMIN_CHANGE_VIEW_REVERSE_NAMES.get(instance.template.origin, "admin:index")
+        reverse_args = [] if reverse_name == "admin:index" else [instance.template.pk]
+
+        url = reverse(reverse_name, args=reverse_args)
         return format_html('<a href="{}">{}</a>', url, instance.template)
 
     template_link.short_description = _("badge template")
 
     def response_change(self, request, obj):
         if "_save" in request.POST:
-            return HttpResponseRedirect(reverse("admin:badges_credlybadgetemplate_change", args=[obj.template.pk]))
+            reverse_name = ADMIN_CHANGE_VIEW_REVERSE_NAMES.get(obj.template.origin, "admin:index")
+            reverse_args = [] if reverse_name == "admin:index" else [obj.template.pk]
+            return HttpResponseRedirect(reverse(reverse_name, args=reverse_args))
         return super().response_change(request, obj)
 
 
@@ -445,7 +456,9 @@ def template_link(self, instance):
         """
         Interactive link to parent (badge template).
         """
-        url = reverse("admin:badges_credlybadgetemplate_change", args=[instance.template.pk])
+        reverse_name = ADMIN_CHANGE_VIEW_REVERSE_NAMES.get(instance.template.origin, "admin:index")
+        reverse_args = [] if reverse_name == "admin:index" else [instance.template.pk]
+        url = reverse(reverse_name, args=reverse_args)
         return format_html('<a href="{}">{}</a>', url, instance.template)
 
     template_link.short_description = _("badge template")
@@ -460,7 +473,9 @@ def formfield_for_manytomany(self, db_field, request, **kwargs):
 
     def response_change(self, request, obj):
         if "_save" in request.POST:
-            return HttpResponseRedirect(reverse("admin:badges_credlybadgetemplate_change", args=[obj.template.pk]))
+            reverse_name = ADMIN_CHANGE_VIEW_REVERSE_NAMES.get(obj.template.origin, "admin:index")
+            reverse_args = [] if reverse_name == "admin:index" else [obj.template.pk]
+            return HttpResponseRedirect(reverse(reverse_name, args=reverse_args))
         return super().response_change(request, obj)
 
 
@@ -562,11 +577,15 @@ def sync_groups(self, request, queryset):
         """
         site = get_current_site(request)
         for api_config in queryset:
-            call_command(
-                "sync_accredible_groups",
-                api_config_id=api_config.id,
-                site_id=site.id,
-            )
+            try:
+                call_command(
+                    "sync_accredible_groups",
+                    api_config_id=api_config.id,
+                    site_id=site.id,
+                )
+            except BadgeProviderError as exc:
+                messages.set_level(request, messages.ERROR)
+                messages.error(request, _("Failed to sync groups for API config: {}. {}").format(api_config.name, exc))
 
         messages.success(request, _("Accredible groups were successfully updated."))
 

credentials/apps/badges/base_api_client.py

@@ -62,8 +62,8 @@ def _raise_for_error(self, response):
         try:
             response.raise_for_status()
         except HTTPError:
-            logger.error(f"Error while processing request: {response.status_code} - {response.text}")
-            raise BadgeProviderError(f"{self.PROVIDER_NAME} API:{response.text}({response.status_code})")
+            logger.error(f"Error while processing {self.PROVIDER_NAME} request: {response.status_code} - {response.text}")
+            raise BadgeProviderError(f"{response.text} Status({response.status_code})")
 
     @property
     def base_api_url(self):