refactor: avoid confusion between Credential and User Credential

credentials/apps/badges/admin.py

@@ -304,7 +304,7 @@ class BadgeProgressAdmin(admin.ModelAdmin):
         "issued",
     )
     exclude = [
-        "credential",
+        "user_credential",
     ]
 
     @admin.display(boolean=True)
@@ -321,7 +321,7 @@ def issued(self, obj):
         """
         Identifies if user credential exists (regardless its current status).
         """
-        return bool(obj.credential)
+        return bool(obj.user_credential)
 
 
 class CredlyBadgeAdmin(admin.ModelAdmin):
@@ -330,18 +330,26 @@ class CredlyBadgeAdmin(admin.ModelAdmin):
     """
 
     list_display = (
+        "uuid",
         "username",
+        "credential",
+        "status",
         "state",
-        "uuid",
+        "external_uuid",
     )
-    list_filter = ("state",)
+    list_filter = ("status", "state",)
     search_fields = (
         "username",
-        "uuid",
+        "external_uuid",
     )
     readonly_fields = (
+        "credential_id",
+        "credential_content_type",
+        "username",
+        "download_url",
         "state",
         "uuid",
+        "external_uuid",
     )
 
 

credentials/apps/badges/migrations/0016_rename_credential_badgeprogress_user_credential.py

@@ -0,0 +1,18 @@
+# Generated by Django 3.2.20 on 2024-04-16 16:01
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('badges', '0015_credlybadge_external_uuid'),
+    ]
+
+    operations = [
+        migrations.RenameField(
+            model_name='badgeprogress',
+            old_name='credential',
+            new_name='user_credential',
+        ),
+    ]

credentials/apps/badges/models.py

@@ -417,7 +417,7 @@ class BadgeProgress(models.Model):
     - user-centric;
     """
 
-    credential = models.OneToOneField(
+    user_credential = models.OneToOneField(
         UserCredential,
         models.SET_NULL,
         blank=True,