enhance: アカウント削除時のクライアントの挙動をいい感じにするなど
Resolve #9322
What
- api/iで、もしuserProfileを検索した結果がnullであればアカウントが削除されているということなので、その旨を明示するエラーを返す
- クライアントのaccount.ts
- fetchAccount (クライアント起動時やユーザー切り替え時に呼ばれる)で、トークン失効時やアカウント削除時にエラーを表示してログアウトするように
- openAccountMenu (ナビゲーションの最下部などを押すと呼ばれるアカウントメニュー)で、もしusers/showでuserIdが見つからなかった場合にはuserIdを表示し、クリックするとログインするように(もしトークン失効などがあれば↑が働く)
- settings/accounts.vueのデザイン変更など
- 型エラー潰し
Why
アカウント周りの挙動が怪しいのをなおしたかった
Codecov Report
Merging #10002 (fe004ed) into develop (a4ca127) will decrease coverage by
0.06%. The diff coverage is15.96%.
@@ Coverage Diff @@
## develop #10002 +/- ##
===========================================
- Coverage 70.80% 70.74% -0.06%
===========================================
Files 807 807
Lines 77207 77286 +79
Branches 5378 5380 +2
===========================================
+ Hits 54666 54678 +12
- Misses 22541 22608 +67
| Impacted Files | Coverage Δ | |
|---|---|---|
| packages/frontend/src/account.ts | 11.40% <3.00%> (-3.07%) |
:arrow_down: |
| packages/backend/src/server/api/ApiCallService.ts | 62.42% <66.66%> (ø) |
|
| packages/backend/src/server/api/endpoints/i.ts | 97.46% <86.66%> (-2.54%) |
:arrow_down: |
| packages/backend/src/server/api/error.ts | 78.57% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
refreshAccountを作り直してログイン時に要求するようにした
死ぬほどアカウント登録してる狂人と重いサーバーの組み合わせでかなり厄介なことになりそうな予感がするので全アカウントrefreshはもうちょっと本当にやるか考えたほうがいいかも (単純計算で10アカウントにログインしていて一回の確認に1秒かかる場合アカウント管理画面を開くのに10秒かかる)
refreshはもうちょっと本当にやるか考えたほうが
なんか要らない気がしてきた(フォールバック表示は実装する必要ある
フォールバック表示とか要らないし"なかったこと"にすればいいか
なんかめっちゃ変えた
lintが落ちとる
はい
👍🏻