Minor tweak to task delegation.

Author: HalfVoxel

api/src/migrations/0042_task_delegation4.sql

@@ -0,0 +1,13 @@
+ALTER TABLE task_delegation_log
+MODIFY COLUMN action ENUM(
+    'assigned',
+    'completed',
+    'not_done_did_something_else',
+    'not_done_confused',
+    'not_done_forgot',
+    'not_done_no_time',
+    'not_done_other',
+    'not_done_rerolled',
+    'ignored',
+    'already_completed_by_someone_else'
+) NOT NULL;

api/src/tasks/delegate.py

@@ -1056,7 +1056,12 @@ def member_recently_received_task(ctx: TaskContext) -> bool:
         # If a member has been assigned (but not necessarily completed) a task recently, don't assign them another one too soon.
         # This can be relevant if a member quickly says that they cannot complete a task for some reason.
         # It's then no longer assigned or completed, but we still don't want to assign them another task too soon.
-        if ctx.time - last_assigned_task.created_at < timedelta(hours=14):
+        #
+        # If the member explicitly wanted a new task, we allow assigning a new task sooner.
+        if (
+            ctx.time - last_assigned_task.created_at < timedelta(hours=14)
+            and last_assigned_task.status != "not_done_rerolled"
+        ):
             return True
 
     return False

api/src/tasks/models.py

@@ -25,6 +25,7 @@ class TaskDelegationLog(Base):
         "not_done_forgot",
         "not_done_no_time",
         "not_done_other",
+        "not_done_rerolled",
         "ignored",
         "already_completed_by_someone_else",
     ]
@@ -52,6 +53,7 @@ class TaskDelegationLog(Base):
             "not_done_forgot",
             "not_done_no_time",
             "not_done_other",
+            "not_done_rerolled",
             "ignored",
             "already_completed_by_someone_else",
             name="task_action",

api/src/tasks/views.py

@@ -373,7 +373,7 @@ def slack_handle_task_feedback(payload: SlackInteraction, ignore_reasons: list[s
                 member.member_id, slack_interaction=None, force=True, ignore_reasons=ignore_reasons
             )
         else:
-            log.action = "not_done_other"  # TODO: Replace by "rerolled" or something
+            log.action = "not_done_rerolled"
             logger.info(f"Member #{member.member_number} {member.firstname} {member.lastname} requested a new task")
             db_session.commit()