設定画面のブラウザタブにあるキャッシュ項目の改修 #4184
設定画面のブラウザタブにあるキャッシュ項目の改修 #4184
Conversation
|
Preview (prod) → https://4184-prod.traq-preview.trapti.tech/ |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4184 +/- ##
==========================================
+ Coverage 86.38% 86.41% +0.02%
==========================================
Files 66 66
Lines 4730 4747 +17
Branches 569 503 -66
==========================================
+ Hits 4086 4102 +16
- Misses 638 639 +1
Partials 6 6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
reiroop
changed the title
reiroop
force-pushed
the
fix/setting_browser_cache
branch
from
37975cd
to
fd53bb8
Compare
スペルミスの修正
各要素の配置を調整 checkboxの見た目が未完成 modalのsubtitleがずれてる
|
conflict対応しました！初めてなので確認お願いします。 |
|
conflict解消は問題なさそうです(なんで発生してたかよく分かってない) |
|
あ、issue紐づけじゃうと残ってる僕のPRがマージされる前にissue closeされちゃうので、紐づけはしないでおいてほしいです🙏 |
了解です |
そうだったんですね、ありがとうございます |
|
キャッシュの問題が片付いたわけではないのですが、僕の能力ではこれ以上は難しいのと、処理の仕様としては以前のものと変わっていないはずなので、issueとして分離して完了としてしまいたいです |
キャッシュサイズを表示しない方向で行きます！ |
| // const cacheSize = ref( | ||
| // Object.fromEntries(cacheCategories.map(name => [name, '計算中...'])) | ||
| // ) | ||
| // const updateCacheSize = async () => { | ||
| // await Promise.all( | ||
| // cacheCategories.map(async category => { | ||
| // cacheSize.value[category] = prettifyFileSize( | ||
| // await calculateCacheSizeSum(await cacheNames(category)) | ||
| // ) | ||
| // }) | ||
| // ) | ||
| // } | ||
| // onMounted(updateCacheSize) | ||
| // const calculateCacheSizeSum = async (cacheNames: string[]) => { | ||
| // let size = 0 | ||
| // await Promise.all( | ||
| // cacheNames.map(async cacheName => { | ||
| // size += await calculateEachCacheSize(cacheName) | ||
| // }) | ||
| // ) | ||
| // return size | ||
| // } | ||
| // const calculateEachCacheSize = async (cacheName: string) => { | ||
| // const cache = await window.caches.open(cacheName) | ||
| // const keys = await cache.keys() | ||
| // let size = 0 | ||
| // await Promise.all( | ||
| // keys.map(async key => { | ||
| // const response = await cache.match(key) | ||
| // if (!response) return | ||
| // const blob = await response.blob() | ||
| // size += blob.size | ||
| // }) | ||
| // ) | ||
| // return size | ||
| // } |
There was a problem hiding this comment.
コメントアウトじゃなくて一旦全部消しちゃっていいと思います
その上で、キャッシュサイズを表示するというissueを別で作っておいて、この部分を消したときのcommitのURLを一緒に貼っておいてほしいです
|
CI落ちてるので修正お願いします 🙏 |
|
しました！ |
Figma: https://www.figma.com/file/6Wme6N24y3GN2jD0vqZWZr/traQ-S-UI?type=design&node-id=9739-23813&mode=design&t=YH2DxrncCa8ICrVt-0