fixup! 目標完了時にCloud Tasksの通知が削除されない問題を修正

MurakawaTakuya · Jan 28, 2025 · c1762fa · c1762fa
1 parent 944458b
commit c1762fa
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/functions/src/index.ts b/functions/src/index.ts
@@ -115,7 +115,7 @@ export const firestore = onRequest({ region: region }, async (req, res) => {
 export {
   createTasksOnGoalCreate,
   deleteTasksOnGoalDelete,
-  updateTasksOnPostUpdate as deleteTasksOnPostCreate,
+  updateTasksOnPostUpdate,
 } from "./tasks";
 
 // テスト用API

diff --git a/functions/src/tasks.ts b/functions/src/tasks.ts
@@ -13,7 +13,7 @@ const region = "asia-northeast1";
 const queue = "deadline-notification-queue";
 const marginTime = 10;
 
-// TODO: タスク名の後ろに日時の値をつけて一意にする
+// queuePath, postData, etaを受け取り、タスクを作成
 const createTask = async (
   queuePath: string,
   postData: { goalId: string; marginTime: number },
@@ -126,13 +126,11 @@ export const updateTasksOnPostUpdate = onDocumentUpdated(
       return;
     }
 
-    logger.debug("Before:", event.data.before.data());
-    logger.debug("After:", event.data.after.data());
-
     if (
       event.data.before.data().post === null &&
       event.data.after.data().post !== null
     ) {
+      // 目標を完了(完了投稿を作成)した場合
       // null -> post に変更された場合はタスクを削除
       try {
         const goalId = event.params.goalId;
@@ -145,7 +143,9 @@ export const updateTasksOnPostUpdate = onDocumentUpdated(
       event.data.before.data().post !== null &&
       event.data.after.data().post === null
     ) {
+      // 完了投稿を削除した場合
       // post -> null に変更された場合はタスクを作成(deadlineが今よりも後の場合のみ)
+
       if (
         event.data.after.data().deadline.toDate().getTime() <=
         new Date().getTime()