Skip to content
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.

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

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

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

MurakawaTakuya
Copy link
Owner

@MurakawaTakuya MurakawaTakuya commented Jan 27, 2025

概要

目標完了時にCloud Tasksの通知が削除されない問題を修正
同時に以下の機能も実装

  • 期限を変更した時に通知送信時間を更新
  • 完了投稿だけを削除した時に通知を再度登録

変更内容

  • 目標完了時にCloud Tasksの通知が削除されない問題を修正
    条件文とパスのミス
      Error deleting task: Error: 5 NOT_FOUND: Requested entity was not found.
    
  • 期限を変更した時に通知送信時間を変更
    期限を変更したら、通知を再設定するためにCloud tasksにHTTPリクエストを再設定しようとしたが以下のエラーのように、同じ名前のtaskをすぐに再作成することができないようだったので、goalIdの後ろに作成時間のDate型の数値をつけることで一意にする。また、削除時はgoalIdで始まるtaskを探して削除するようにした。
    Exception from a finished function: Error: 6 ALREADY_EXISTS: The task cannot be created because a task with this name existed too recently. For more information about task de-duplication see https://cloud.google.com/tasks/docs/reference/rest/v2/projects.locations.queues.tasks/create#body.request_body.FIELDS.task.
    
    image

現状目標と完了投稿の状態が変わった場合の処理は次の通り

  • 目標

    • 作成: createTasksOnGoalCreateでtask作成
    • 削除: deleteTasksOnGoalDeleteでtask削除
    • 期限変更: updateTasksOnPostUpdateでtask削除・作成
  • 完了投稿

    • 作成: updateTasksOnPostUpdateでtask削除
    • 削除: updateTasksOnPostUpdateでtask作成
  • deleteTasksOnPostCreateupdateTasksOnPostUpdateに変更した

確認方法

チェックリスト

  • PRに必要な内容を記述し、Assignやタイトルを設定した
  • File Changedで不要な変更や誤った変更が無いか確認した
  • 対応したissueをProjectの"レビュー中・待機中"に移動した
  • レビューを頼んだ人にDiscordでお願いした
  • 機密情報が含まれていないことを確認した(含まれている場合は直ちにリモートからコミットを削除すること)
  • UIを変更した場合
    • PCで見た時に表示が崩れていないことを確認した
    • スマホでも正常に表示されることを確認した(大きい画面と小さい画面両方で)

@MurakawaTakuya MurakawaTakuya self-assigned this Jan 27, 2025
@MurakawaTakuya MurakawaTakuya marked this pull request as ready for review January 28, 2025 09:53
@MurakawaTakuya MurakawaTakuya force-pushed the fix/172-fix-delete-tasks-on-post-create branch from c1762fa to ff6bec2 Compare January 28, 2025 10:09
@MurakawaTakuya MurakawaTakuya force-pushed the fix/172-fix-delete-tasks-on-post-create branch from ff6bec2 to 8b49326 Compare January 28, 2025 11:32
@MurakawaTakuya MurakawaTakuya merged commit 7aee594 into main Jan 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

目標完了時にCloud Tasksの通知送信が削除されていない
1 participant