Skip to content

Conversation

@aster-void
Copy link
Contributor

@aster-void aster-void commented Sep 30, 2024

PRの概要

closes #302
useSWR を Friend ページと Chat ページに適用

具体的な変更内容

Friend + Chat。
Settings の Course は難しくて一旦パス。
Home は SWR と相性が悪い (すると Twitter みたいに出てきた人がすぐ流れるようになる) ので Skeleton UI などにする予定。
あとなんか Home 触ってたら #301#296 も直ったので、 close #301 and close #296

影響範囲

Home 以外すべてのページとHome。

動作要件

なし。

レビューリクエストを出す前にチェック!

  • 改めてセルフレビューしたか
  • 手動での動作検証を行ったか
  • server の機能追加ならば、テストを書いたか
    • 理由: server の機能追加ではない
  • 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
    • 理由: 間違った使い方は存在しない
  • わかりやすいPRになっているか

@aster-void aster-void changed the title キャッシュによる UX の改善 SWR キャッシュによる UX の改善 Sep 30, 2024
@aster-void aster-void marked this pull request as ready for review September 30, 2024 07:24
@aster-void aster-void requested review from GuY8528, KaichiManabe and nakaterm and removed request for GuY8528, KaichiManabe and nakaterm September 30, 2024 07:25
@aster-void aster-void added feature New feature or request optimization labels Sep 30, 2024
@aster-void aster-void mentioned this pull request Sep 30, 2024
2 tasks
Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

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

なんかいい気がするが、夜中に見てもよく分からん...

@aster-void aster-void changed the title SWR キャッシュによる UX の改善 SWR キャッシュによる UX の改善となんか挙動に文句言われたので SQL 書き直した Oct 3, 2024
Copy link
Contributor

@KaichiManabe KaichiManabe left a comment

Choose a reason for hiding this comment

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

多分いいと思います!

@aster-void aster-void merged commit c82e424 into main Oct 5, 2024
@aster-void aster-void deleted the nice-cache branch October 5, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

3 participants