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

chore: cacheRemoteFilesのデフォルト値をfalseに #11398

Merged
merged 8 commits into from
Aug 5, 2023

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Jul 27, 2023

Fix #10043

What

  • cacheRemoteFilesのデフォルト値をfalseに
  • cacheRemoteFilesの説明文を変更
  • マイグレーションのゴミを変更

image

Why

cacheRemoteFilesは基本的にオフで良い

サムネイルが表示されないという説明は誤り(プロキシ使用オプションがあった時代の名残か?)

マイグレーションのゴミを取り除きたかった

Additional info (optional)

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/backend Server side specific issue/PR label Jul 27, 2023
@syuilo
Copy link
Member

syuilo commented Jul 27, 2023

ファイル管理の🗑️ボタンからリモートファイルのキャッシュを削除できます。

は書くとしたら別のキーにしたい感

@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #11398 (dd7963a) into develop (c5b8766) will increase coverage by 15.48%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop   #11398       +/-   ##
============================================
+ Coverage    78.45%   93.93%   +15.48%     
============================================
  Files          174        2      -172     
  Lines        22658      462    -22196     
  Branches       500       59      -441     
============================================
- Hits         17777      434    -17343     
+ Misses        4881       28     -4853     

see 172 files with indirect coverage changes

@syuilo
Copy link
Member

syuilo commented Jul 27, 2023

(長くなるし、今後キャッシュクリアのUIが改修される可能性が高く、そうした場合

この設定を有効にすると、リモートファイルをこのサーバーのストレージにキャッシュするようになります。画像の表示が高速化されますが、サーバーのストレージを多く消費するようになります。この設定が無効の場合、画像のサムネイル生成やユーザーのプライバシー保護のために、default.ymlでproxyRemoteFilesをtrueにすることをお勧めします。

部分の翻訳までリセットされそう)

@tamaina
Copy link
Contributor Author

tamaina commented Jul 27, 2023

別キーにした

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jul 27, 2023
@tamaina tamaina marked this pull request as ready for review July 27, 2023 03:18
@tamaina
Copy link
Contributor Author

tamaina commented Jul 27, 2023

(あ、readyにしてなかった

@github-actions github-actions bot requested a review from acid-chicken July 27, 2023 03:18
@danominium
Copy link
Contributor

設定の変更はCHANGELOGへ残して頂けるとありがたいです。
更新時に気づき易くなるので。

@syuilo syuilo merged commit 2217a3c into misskey-dev:develop Aug 5, 2023
@syuilo
Copy link
Member

syuilo commented Aug 5, 2023

👍🏻

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