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

Fix(frontend): プロフィール編集後にプロフィール画面に行ってもリロードをしないと編集内容が反映されない。 #13118

Conversation

mesichicken
Copy link

What

バグ修正です。
プロフィール編集後にプロフィール画面に行ってもリロードをしないと編集内容が反映されない問題を修正しました。

Why

本バグが存在するとユーザーはプロフィール編集をしたのに、反映がされていないように感じてしまう。

Additional info (optional)

#13078

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 31, 2024
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9ac2c36) 64.22% compared to head (9ea678f) 77.52%.
Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13118       +/-   ##
============================================
+ Coverage    64.22%   77.52%   +13.29%     
============================================
  Files          976      183      -793     
  Lines       108707    24649    -84058     
  Branches      5582      462     -5120     
============================================
- Hits         69813    19108    -50705     
+ Misses       38894     5541    -33353     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mesichicken mesichicken force-pushed the fix/profile-update-without-reload branch from b9c8c3d to 9ea678f Compare January 31, 2024 06:48
immediate: true,
deep: true,
},
);
Copy link
Author

Choose a reason for hiding this comment

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

$i(accountData)を監視対象に追加することで、ユーザー情報が編集されたらfetchUserするようにしました。

@mesichicken mesichicken marked this pull request as ready for review January 31, 2024 06:54
@mesichicken mesichicken changed the title Fix(frontend)プロフィール編集後にプロフィール画面に行ってもリロードをしないと編集内容が反映されない。 Fix(frontend): プロフィール編集後にプロフィール画面に行ってもリロードをしないと編集内容が反映されない。 Jan 31, 2024
@syuilo
Copy link
Member

syuilo commented Feb 1, 2024

今後パフォーマンス上リロードしたからといって変更が即時反映されるとの保証はされなくなることも考えられるから、例えばGitHubのように「変更したとしてもすぐ反映されるわけではない」ということをユーザーに伝えるようにするアプローチの方が良さそうに思いました

@syuilo
Copy link
Member

syuilo commented Feb 1, 2024

あと$i全体をwatchするのはちょっと危ないかも?
するとしたらnameとかプロフィールで使われるプロパティだけにするとか

@samunohito
Copy link
Member

パフォーマンス上

具体的にどのようなことを懸念されていますか?
プロフィールは頻繁に変わるものではない上、変わったとしても1回だけユーザ情報を再読み込みするだけなので、そこまで重くならないかと思っていましたが…

@kakkokari-gtyih
Copy link
Contributor

今後別のページの強制リロード(戻ったときに更新されといてほしい)にも再利用したいので #13078 (comment) のやり方のほうが良いかも?(と思ったがKeepAliveのキャッシュを削除するための方法が提供されていないらしい)

@kakkokari-gtyih
Copy link
Contributor

KeepAliveのキャッシュを削除するための方法が提供されていないらしい

気合で実装した #13180

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Feb 6, 2024

Done in #13180
(すみません)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants