misskey icon indicating copy to clipboard operation
misskey copied to clipboard

enhance: アカウント削除時のクライアントの挙動をいい感じにするなど

Open tamaina opened this issue 2 years ago • 6 comments

Resolve #9322

What

  • api/iで、もしuserProfileを検索した結果がnullであればアカウントが削除されているということなので、その旨を明示するエラーを返す
  • クライアントのaccount.ts
    • fetchAccount (クライアント起動時やユーザー切り替え時に呼ばれる)で、トークン失効時やアカウント削除時にエラーを表示してログアウトするように
    • openAccountMenu (ナビゲーションの最下部などを押すと呼ばれるアカウントメニュー)で、もしusers/showでuserIdが見つからなかった場合にはuserIdを表示し、クリックするとログインするように(もしトークン失効などがあれば↑が働く)
  • settings/accounts.vueのデザイン変更など
  • 型エラー潰し

Why

アカウント周りの挙動が怪しいのをなおしたかった

tamaina avatar Feb 19 '23 17:02 tamaina

Codecov Report

Merging #10002 (fe004ed) into develop (a4ca127) will decrease coverage by 0.06%. The diff coverage is 15.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.

codecov[bot] avatar Feb 19 '23 17:02 codecov[bot]

refreshAccountを作り直してログイン時に要求するようにした

tamaina avatar Mar 01 '23 11:03 tamaina

死ぬほどアカウント登録してる狂人と重いサーバーの組み合わせでかなり厄介なことになりそうな予感がするので全アカウントrefreshはもうちょっと本当にやるか考えたほうがいいかも (単純計算で10アカウントにログインしていて一回の確認に1秒かかる場合アカウント管理画面を開くのに10秒かかる)

rinsuki avatar Mar 05 '23 11:03 rinsuki

refreshはもうちょっと本当にやるか考えたほうが

なんか要らない気がしてきた(フォールバック表示は実装する必要ある

tamaina avatar Mar 05 '23 11:03 tamaina

フォールバック表示とか要らないし"なかったこと"にすればいいか

tamaina avatar Mar 05 '23 12:03 tamaina

なんかめっちゃ変えた

tamaina avatar Mar 05 '23 13:03 tamaina

lintが落ちとる

syuilo avatar Mar 08 '23 23:03 syuilo

はい

tamaina avatar Mar 09 '23 05:03 tamaina

👍🏻

syuilo avatar Mar 09 '23 05:03 syuilo