Skip to content
This repository has been archived by the owner on Dec 12, 2023. It is now read-only.

fix: Useが残っている洗濯機を使用開始した際の問題を修正 #82

Merged
merged 3 commits into from
Oct 6, 2023

Conversation

tufusa
Copy link
Collaborator

@tufusa tufusa commented Oct 5, 2023

Close #81

主な変更

  • 以前は、Useが残っている洗濯機を使用した場合、アプリでの洗濯開始時に「終了後のUseが残っていたら消す」としていました。
  • この方法だと、アプリを使用しない人が次に洗濯機を使った場合Useが残ってしまい、以前の使用者のHome画面に"洗濯中"が表示されてしまうという問題がありましたら。
  • よって、洗濯機の状態更新時に「終了後のUseが残っていたら消す」という処理を行うことで、アプリの使用にかかわらずUseを更新できるようにしました。
    • 「誰かによって取り出されました」という旨の通知も発行されるようにしました。

付随する修正

  • Home画面の自動リフレッシュで、洗濯機の空き状況は更新されていましたが、useRevalidatorの仕様上自分の使用状況は更新されていませんでした。
  • これを修正し、revalidate時に自分の使用状況も更新されるようになりました。

Copy link
Collaborator

@speak-mentaiko speak-mentaiko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

追加された通知メッセージの 取り出されたようです。というところが少し気になります。

@tufusa
Copy link
Collaborator Author

tufusa commented Oct 5, 2023

@speak-mentaiko

追加された通知メッセージの 取り出されたようです。というところが少し気になります。

どのような違和感を覚えましたか?適切な文言が思いつかなかったので教えてくれると嬉しいです。

@speak-mentaiko
Copy link
Collaborator

取り出されたようです。だと推量の域を出ないので取り出されました。などはどうでしょうか?

@tufusa
Copy link
Collaborator Author

tufusa commented Oct 5, 2023

なるほどです!取り忘れ通知の文面に合わせたつもりでしたが、それより確実性が高いので断定の方がいいですかね

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 5, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 40d49d6
Status: ✅  Deploy successful!
Preview URL: https://6daedf97.washboard.pages.dev
Branch Preview URL: https://fix-laundry-re-start.washboard.pages.dev

View logs

@tufusa tufusa merged commit 4c850b0 into main Oct 6, 2023
3 checks passed
Copy link
Contributor

@kiharu3112 kiharu3112 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants