-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: センシティブな画像をアイコン・バナーに指定できないように #13676
base: develop
Are you sure you want to change the base?
Conversation
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -50326,6 +50326,24 @@
}
}
},
+ "AVATAR_IS_SENSITIVE": {
+ "value": {
+ "error": {
+ "message": "The file specified as an avatar is marked as sensitive.",
+ "code": "AVATAR_IS_SENSITIVE",
+ "id": "71bb5e53-4742-4609-b465-36081e131208"
+ }
+ }
+ },
+ "BANNER_IS_SENSITIVE": {
+ "value": {
+ "error": {
+ "message": "The file specified as a banner is marked as sensitive.",
+ "code": "BANNER_IS_SENSITIVE",
+ "id": "e148b34c-9f33-4300-93e0-7817008fb366"
+ }
+ }
+ },
"NO_SUCH_PAGE": {
"value": {
"error": { |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13676 +/- ##
============================================
- Coverage 77.90% 65.36% -12.55%
============================================
Files 185 1005 +820
Lines 25563 115331 +89768
Branches 487 4629 +4142
============================================
+ Hits 19916 75387 +55471
- Misses 5640 39917 +34277
- Partials 7 27 +20 ☔ View full report in Codecov by Sentry. |
/preview |
/preview |
プレビューどうやって見るんだっけ |
/preview |
「アイコン画像を変更」などを押しても何も起こらないわね |
たぶん治ったと思われる |
/preview |
コンフリクト解消 |
/preview |
(これエラー処理のアラート掲出を別関数に追いやるんじゃなくてos.apiWithDialogにカスタムエラーを指定できるような仕組みにしたほうが良いかも) |
done |
/prevoew |
/preview |
selectFile関数は名前通りファイルを選択させるためのAPIだから、そこに追加でメニューを設定してそれ以外のアクション(アイコン/バナー解除)できるようにするというのは設計上微妙な感じがある |
とはいえ良い代替案は思いつかない |
selectFile自体にその機能を設けてるわけではないので別にいい気はする |
この仕様では「デフォルトでメディアをセンシティブ設定にする」を設定している人間が、プロフィールからアイコンをアップロードして変更ができなくなってしまうのだわね。 |
たぶんロールで「ファイルにNSFWを常に付与」にされたユーザーは、もうアイコンを変更できなくなってしまう。 |
かといってセンシティブがついている画像を指定できてしまうのもそれはそれで問題だとは思うけど… |
まあ外すことはできるようになったのでいいんじゃないかしら(「ファイルにNSFWを常に付与」を付与される人って基本的に何らかのペナルティを食らっている人のはずなので…?) |
コンフリクト解消 |
コンフリクト解消 |
これってリモートユーザーに対しても有効ですか? |
自分をアップデートするAPIの動作内で止めてるだけなのでそんなことはないはず |
What
i/update
でセンシティブな画像の場合はエラーで弾くようにWhy
Fix #7311
Additional info (optional)
os.apiWithDialog
でエラーIDをもとにカスタムのエラーダイアログが出せるようになりました( #14069 の型無し版)Checklist