From efb2da8adbdb5164d9b03c3ecf0015b8e1bb21dc Mon Sep 17 00:00:00 2001
From: Edward Keeble <edward@developmentseed.org>
Date: Wed, 7 Feb 2024 15:31:14 -0400
Subject: [PATCH 1/3] Use content_type to group dependents, since view_model is
 None.

---
 app/admin_ui/templates/snippets/dependent_drafts.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/admin_ui/templates/snippets/dependent_drafts.html b/app/admin_ui/templates/snippets/dependent_drafts.html
index fc7e7d50a..097fc7ee4 100644
--- a/app/admin_ui/templates/snippets/dependent_drafts.html
+++ b/app/admin_ui/templates/snippets/dependent_drafts.html
@@ -1,7 +1,7 @@
-{% regroup descendents by view_model as grouped_descendents %}
+{% regroup descendents by content_type as grouped_descendents %}
 <ul class="list-unstyled">
   {% for model_type in grouped_descendents %}
-  <li class="font-weight-bold">{{ model_type.grouper }}</li>
+  <li class="font-weight-bold">{{ model_type.grouper.model | capfirst }}</li>
   <ul>
       {% for instance in model_type.list %}
       <li>

From 145225d7c0eef4eb66df7f5079e536edd90c24cd Mon Sep 17 00:00:00 2001
From: Edward Keeble <edward@developmentseed.org>
Date: Thu, 8 Feb 2024 09:57:43 -0400
Subject: [PATCH 2/3] Run distinct on canonical uuid of dependent change
 objects, rather than on model_instance_uuid, since NULL values are treated as
 equal by distinct. This was preventing us from listing multiple unpublished
 dependent objects, since model_instance_uuid on each was NULL.

---
 app/admin_ui/views/v2.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/admin_ui/views/v2.py b/app/admin_ui/views/v2.py
index c42c075ba..5f9ea77a2 100644
--- a/app/admin_ui/views/v2.py
+++ b/app/admin_ui/views/v2.py
@@ -12,6 +12,7 @@
 from django.contrib.contenttypes.models import ContentType
 from django.views.generic.edit import UpdateView
 from django.db.models import OuterRef, Subquery
+from django.db.models.functions import Coalesce
 from django.views.generic.edit import CreateView
 from django_filters.views import FilterView
 from django_tables2 import SingleTableView, SingleTableMixin
@@ -651,8 +652,13 @@ def get_object(self, queryset=None):
     def _filter_latest_changes(change_queryset):
         """Returns the single latest Change draft for each model_instance_uuid in the
         provided queryset."""
-        return change_queryset.order_by('model_instance_uuid', '-approvallog__date').distinct(
-            'model_instance_uuid'
+        # Using Coalesce to handle the case where model_instance_uuid is None,
+        # since distinct will treat NULLs as equal.
+        # Using `cid` rather than `canonical_uuid` to avoid a name conflict on the model
+        return (
+            change_queryset.annotate(cid=Coalesce("model_instance_uuid", "uuid"))
+            .order_by('cid', '-approvallog__date')
+            .distinct('cid')
         )
 
     def get_context_data(self, **kwargs):

From d918654e3ce6aae812df338e181dab0e5b38a9dc Mon Sep 17 00:00:00 2001
From: Lint Action <lint-action@samuelmeuli.com>
Date: Thu, 8 Feb 2024 14:01:57 +0000
Subject: [PATCH 3/3] Fix code style issues with Black

---
 .../contrib/sites/migrations/0003_set_site_domain_and_name.py  | 1 +
 app/admin_ui/admin/data_models.py                              | 3 +--
 app/config/urls.py                                             | 2 +-
 app/config/wsgi.py                                             | 1 +
 4 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/admg_webapp/contrib/sites/migrations/0003_set_site_domain_and_name.py b/app/admg_webapp/contrib/sites/migrations/0003_set_site_domain_and_name.py
index ecd4b69a7..efdd28063 100644
--- a/app/admg_webapp/contrib/sites/migrations/0003_set_site_domain_and_name.py
+++ b/app/admg_webapp/contrib/sites/migrations/0003_set_site_domain_and_name.py
@@ -3,6 +3,7 @@
 
 http://cookiecutter-django.readthedocs.io/en/latest/faq.html#why-is-there-a-django-contrib-sites-directory-in-cookiecutter-django
 """
+
 from django.conf import settings
 from django.db import migrations
 
diff --git a/app/admin_ui/admin/data_models.py b/app/admin_ui/admin/data_models.py
index 8a15e014f..4df406b52 100644
--- a/app/admin_ui/admin/data_models.py
+++ b/app/admin_ui/admin/data_models.py
@@ -30,8 +30,7 @@
 @admin.register(models.GcmdPhenomenon)
 @admin.register(models.WebsiteType)
 @admin.register(models.Website)
-class BasicAdmin(admin.ModelAdmin, EnforcedPermissionsMixin):
-    ...
+class BasicAdmin(admin.ModelAdmin, EnforcedPermissionsMixin): ...
 
 
 @admin.register(models.CollectionPeriod)
diff --git a/app/config/urls.py b/app/config/urls.py
index a8cbb5a44..45f64faea 100644
--- a/app/config/urls.py
+++ b/app/config/urls.py
@@ -14,7 +14,7 @@
     path("accounts/", include("allauth.urls")),
     path("api/", include("api_app.urls")),
     path("authenticate/token/", CustomTokenView.as_view(), name="token"),
-    path("authenticate/", include("oauth2_provider.urls"), name="oauth2_provider")
+    path("authenticate/", include("oauth2_provider.urls"), name="oauth2_provider"),
     # Your stuff: custom urls includes go here
 ] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
 
diff --git a/app/config/wsgi.py b/app/config/wsgi.py
index 21506485b..0f2fb5500 100644
--- a/app/config/wsgi.py
+++ b/app/config/wsgi.py
@@ -13,6 +13,7 @@
 framework.
 
 """
+
 import os
 import sys