設定画面のブラウザタブにあるキャッシュ項目の改修

[reiroop]

Figma: https://www.figma.com/file/6Wme6N24y3GN2jD0vqZWZr/traQ-S-UI?type=design&node-id=9739-23813&mode=design&t=YH2DxrncCa8ICrVt-0

[github-actions]

Preview (prod) → https://4184-prod.traq-preview.trapti.tech/
Preview (dev) → https://4184-dev.traq-preview.trapti.tech/

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 86.34%. Comparing base (4d63db6) to head (504382d).

Files	Patch %	Lines
src/lib/basic/file.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4184      +/-   ##
==========================================
- Coverage   86.36%   86.34%   -0.03%     
==========================================
  Files          66       66              
  Lines        4722     4722              
  Branches      565      566       +1     
==========================================
- Hits         4078     4077       -1     
- Misses        638      639       +1     
  Partials        6        6              

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

[mehm8128]

プレビューが落ちてるやつ、masterをrebaseとかmasterとかしたら直るはずなのでお願いします:pray:

[mehm8128]

本番環境でも'traQ_S-precache'のkeysがどの環境でも存在しない

/src/sw/workbox.tsがなんか関係してそうってことくらいしか分からないですね...

キャッシュデータの合計が/settings/browserに表示されてるnavigator.storage.estimate()の値とずれているのが気になります

前に言われてた、ちょっと値が違うってやつが蓄積して大きくずれてる気がします
諦めて件数だけ表示(もしくはなんも表示しない)にするか、ずれが生じる原因からちゃんと調べるのどっちかな気がします

[reiroop]

ModalFrameの左右のpaddingを16pxにしたことに関連して、依存するモーダルの修正を考えていたのですが、Figmaを見る限り左右のpaddingが16pxのものと24pxのものと混在していて、かつ最新のデザインはどれを見ればいいのかもよくわからなかったのですが、どのモーダルを修正すればいいですか？
関連:
#4184 (comment)

[mehm8128]

キャッシュのモーダルのpaddingを24pxにしても特に問題なさそうであれば24pxに合わせる方がいい気がしてきたのでそうしてほしいです(つまり、-8pxの指定を消して、ModalFrame側はいじらない)
ちなみに16pxに合わせるとしても既存のモーダルが見た目崩れてなければ特になんか修正する必要もないと思います

[reiroop]

モーダルのpaddingは元に戻しました。
キャッシュサイズの計算や削除の処理はよくわからなかったので、前の仕様に準じました。
おそらくどの環境でも'traQ_S-precache'には何も入っていなくて、'workbox-precache'で始まる種類のキャッシュ(キャッシュオリジン？)にかなり大きいキャッシュが格納されてるらしいというのはわかりました。おそらくこれが値の差の原因だと思うのですが、実態を調べきれなかったのと元のissueの趣旨から離れるので一旦諦めます。
今の処理では'workbox-precache'の削除はしていないので、キャッシュの削除が不十分な可能性があるかもです。

[mehm8128]

キャッシュの合計値がずれてるのはやっぱり気になるので、詳しそうな人にもうちょっと聞いてほしいです...
あと、計算中のときにサイズが何も表示されない状態になってしまっているので、計算週のときは「計算中...」みたいな文字列を入れるとかできるとよさそうです

[mehm8128]

あ、あとconflictの解消もお願いします

[reiroop]

conflict対応しました！初めてなので確認お願いします。
あとCI落ちてる原因がよくわからないので見てもらいたいです。お願いします。

[mehm8128]

conflict解消は問題なさそうです(なんで発生してたかよく分かってない)
CIは多分github落ちてたのが原因で落ちてたのでre-runしたら直りました

[mehm8128]

あ、issue紐づけじゃうと残ってる僕のPRがマージされる前にissue closeされちゃうので、紐づけはしないでおいてほしいです🙏

[reiroop]

あ、issue紐づけじゃうと残ってる僕のPRがマージされる前にissue closeされちゃうので、紐づけはしないでおいてほしいです🙏

了解です

[reiroop]

conflict解消は問題なさそうです(なんで発生してたかよく分かってない) CIは多分github落ちてたのが原因で落ちてたのでre-runしたら直りました

そうだったんですね、ありがとうございます