From 6c8a3d4ddec59146b2e3ee134b06de715e7f3535 Mon Sep 17 00:00:00 2001
From: Raquel Smith <raquelmsmith@users.noreply.github.com>
Date: Mon, 30 Dec 2024 10:33:44 -0800
Subject: [PATCH] exclude playlists with no name and no derived name

---
 posthog/tasks/periodic_digest.py           | 19 +++++++++++++++----
 posthog/tasks/test/test_periodic_digest.py | 15 ++++++---------
 2 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/posthog/tasks/periodic_digest.py b/posthog/tasks/periodic_digest.py
index 136c65a171101..c1d9e020f0a94 100644
--- a/posthog/tasks/periodic_digest.py
+++ b/posthog/tasks/periodic_digest.py
@@ -66,10 +66,21 @@ def get_teams_with_new_event_definitions(end: datetime, begin: datetime) -> Quer
 
 
 def get_teams_with_new_playlists(end: datetime, begin: datetime) -> QuerySet:
-    return SessionRecordingPlaylist.objects.filter(
-        created_at__gt=begin,
-        created_at__lte=end,
-    ).values("team_id", "name", "short_id", "derived_name")
+    return (
+        SessionRecordingPlaylist.objects.filter(
+            created_at__gt=begin,
+            created_at__lte=end,
+        )
+        .exclude(
+            name__isnull=True,
+            derived_name__isnull=True,
+        )
+        .exclude(
+            name="",
+            derived_name="",
+        )
+        .values("team_id", "name", "short_id", "derived_name")
+    )
 
 
 def get_teams_with_new_experiments_launched(end: datetime, begin: datetime) -> QuerySet:
diff --git a/posthog/tasks/test/test_periodic_digest.py b/posthog/tasks/test/test_periodic_digest.py
index e492e1ef95658..f03da75905224 100644
--- a/posthog/tasks/test/test_periodic_digest.py
+++ b/posthog/tasks/test/test_periodic_digest.py
@@ -49,16 +49,17 @@ def test_periodic_digest_report(self, mock_capture: MagicMock) -> None:
                 team=self.team,
                 name="Test Playlist",
             )
-            # These should be included in the digest but use the derived name
-            derived_playlist = SessionRecordingPlaylist.objects.create(
+            # This should be excluded from the digest because it has no name and no derived name
+            SessionRecordingPlaylist.objects.create(
                 team=self.team,
                 name=None,
-                derived_name="Derived Playlist",
+                derived_name=None,
             )
-            derived_playlist_2 = SessionRecordingPlaylist.objects.create(
+            # This should be included in the digest but use the derived name
+            derived_playlist = SessionRecordingPlaylist.objects.create(
                 team=self.team,
                 name="",
-                derived_name="Derived Playlist 2",
+                derived_name="Derived Playlist",
             )
 
             # Create experiments
@@ -170,10 +171,6 @@ def test_periodic_digest_report(self, mock_capture: MagicMock) -> None:
                     "name": "Derived Playlist",
                     "id": derived_playlist.short_id,
                 },
-                {
-                    "name": "Derived Playlist 2",
-                    "id": derived_playlist_2.short_id,
-                },
             ],
             "new_experiments_launched": [
                 {