Fix : Cancel notification with notif manager

app/modules/cinema/endpoints_cinema.py

@@ -149,7 +149,10 @@ async def create_session(
             action_module="cinema",
         )
 
-        await scheduler.cancel_job(job_id=f"cinema_weekly_{sunday}")
+        await notification_tool.cancel_notification(
+            scheduler=scheduler,
+            job_id=f"cinema_weekly_{sunday}",
+        )
 
         await notification_tool.send_notification_to_topic(
             custom_topic=CustomTopic(topic=Topic.cinema),

app/modules/loan/endpoints_loan.py

@@ -828,6 +828,7 @@ async def return_loan(
     db: AsyncSession = Depends(get_db),
     user: models_core.CoreUser = Depends(is_user_a_member),
     scheduler: Scheduler = Depends(get_scheduler),
+    notification_tool: NotificationTool = Depends(get_notification_tool),
 ):
     """
     Mark a loan as returned. This will update items availability.
@@ -872,7 +873,10 @@ async def return_loan(
         returned=True,
         returned_date=datetime.now(UTC),
     )
-    await scheduler.cancel_job(f"loan_end_{loan.id}")
+    await notification_tool.cancel_notification(
+        scheduler=scheduler,
+        job_id=f"loan_end_{loan.id}",
+    )
 
 
 @module.router.post(
@@ -927,9 +931,10 @@ async def extend_loan(
         loan_update=loan_update,
         db=db,
     )
-    # TODO
-
-    await scheduler.cancel_job(f"loan_end_{loan.id}")
+    await notification_tool.cancel_notification(
+        scheduler=scheduler,
+        job_id=f"loan_end_{loan.id}",
+    )
 
     message = Message(
         title="📦 Prêt prolongé",

app/utils/communication/notifications.py

@@ -471,3 +471,10 @@ async def send_future_notification_to_topic_time_defer(
             job_id=job_id,
             defer_seconds=defer_seconds,
         )
+
+    async def cancel_notification(
+        self,
+        scheduler: Scheduler,
+        job_id: str,
+    ):
+        await scheduler.cancel_job(job_id=job_id)