Skip to content

fix(recap): Update associate_related_instances check for better logs #7212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
albertisfu merged 3 commits into from
Apr 10, 2026
+2 −2
Merged

fix(recap): Update associate_related_instances check for better logs #7212

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0062f29
fix(recap): Update associate_related_instances check for better logs
MorganBennetDev Apr 10, 2026
9de0a8a
fix(recap): PR feedback
MorganBennetDev Apr 10, 2026
8683cf5
Merge branch 'main' into morgan/related-instances-log-tweak
albertisfu Apr 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions cl/recap/tasks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -282,9 +282,9 @@ async def associate_related_instances(
"""

if isinstance(pq, EmailProcessingQueue):
if not rd_id:
if rd_id is None:
logger.error(
"rd_id must be a list when pq is EmailProcessingQueue."
"rd_id must not be None when pq is EmailProcessingQueue."
)
return
if isinstance(rd_id, list):
Expand Down
Loading