Update conditions for migrating old automated emails

The condition didn't respect RQJobs in "scheduled" state.
carpentries · Oct 26, 2024 · 88b7d2a · 88b7d2a
1 parent 385d1a7
commit 88b7d2a
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/amy/emails/management/commands/create_emails_for_existing_events.py b/amy/emails/management/commands/create_emails_for_existing_events.py
@@ -64,7 +64,11 @@ def _check_conditions(
         logger.info(f"RQJob for {event=}, {trigger_action=}: {rqjob}")
 
         strategy_result = strategy(event)
-        return strategy_result == StrategyEnum.CREATE and not rqjob
+        return bool(
+            strategy_result == StrategyEnum.CREATE
+            and rqjob
+            and rqjob.status == "scheduled"
+        ) or bool(strategy_result == StrategyEnum.CREATE and not rqjob)
 
     @override_settings(FLAGS={"EMAIL_MODULE": [("boolean", True)]})
     @transaction.atomic