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

fix:ドライブ ファイルの削除を確認するダイアログが誤解を招く #11465

Merged
merged 2 commits into from
Aug 6, 2023

Conversation

193ryo
Copy link
Contributor

@193ryo 193ryo commented Aug 5, 2023

What

#8370の問題点を修正しました。

ドライブのファイル削除時のメッセージ(修正前)
「"ファイル「{name}」を削除しますか?このファイルを使用した全てのコンテンツからも削除されます。"」

修正後
「"ファイル「{name}」を削除しますか?このファイルが添付されたノートからは、ファイルのみ削除されます。(ノートは削除されません。)"」

Why

ファイルは削除ノート自体の削除は行われないことを明記しました。
私のプルリクエストおよびissueへの対応に関する一連の流れに指摘、問題点があれば、遠慮なくご指摘ください。

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

@codecov
Copy link

codecov bot commented Aug 5, 2023

Codecov Report

Merging #11465 (15ee4de) into develop (b439067) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #11465      +/-   ##
===========================================
- Coverage    78.61%   78.61%   -0.01%     
===========================================
  Files          920      920              
  Lines        97388    97388              
  Branches      7695     7694       -1     
===========================================
- Hits         76563    76560       -3     
- Misses       20825    20828       +3     

see 1 file with indirect coverage changes

@syuilo
Copy link
Member

syuilo commented Aug 5, 2023

ノートからはファイルのみ消えるけど、そのほかのコンテンツからはコンテンツまとめて消える可能性がある

@syuilo
Copy link
Member

syuilo commented Aug 5, 2023

ファイル「{name}」を削除しますか?このファイルを使用した一部のコンテンツも消えます。

とかの方がよさそう

@193ryo
Copy link
Contributor Author

193ryo commented Aug 5, 2023

ファイル「{name}」を削除しますか?このファイルを使用した一部のコンテンツも消えます。
↑ありがとうございます!こちらに変更します。

@193ryo
Copy link
Contributor Author

193ryo commented Aug 5, 2023

ファイル「{name}」を削除しますか?このファイルを使用した一部のコンテンツも削除されます。
↑「削除」に平仄を合わせようかと

@syuilo syuilo merged commit 2fb31a5 into misskey-dev:develop Aug 6, 2023
16 checks passed
@syuilo
Copy link
Member

syuilo commented Aug 6, 2023

🙏🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants