-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: デフォルトでフォローするユーザーを指定できるように #14769
base: develop
Are you sure you want to change the base?
Conversation
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -175,6 +175,18 @@
"type": "string"
}
},
+ "defaultFollowedUsers": {
+ "type": "array",
+ "items": {
+ "type": "string"
+ }
+ },
+ "forciblyFollowedUsers": {
+ "type": "array",
+ "items": {
+ "type": "string"
+ }
+ },
"hiddenTags": {
"type": "array",
"items": {
@@ -609,6 +621,8 @@
"translatorAvailable",
"mediaSilencedHosts",
"pinnedUsers",
+ "defaultFollowedUsers",
+ "forciblyFollowedUsers",
"hiddenTags",
"blockedHosts",
"sensitiveWords",
@@ -13768,6 +13782,26 @@
"type": "string"
}
},
+ "defaultFollowedUsers": {
+ "type": [
+ "array",
+ "null"
+ ],
+ "items": {
+ "type": "string",
+ "format": "misskey:id"
+ }
+ },
+ "forciblyFollowedUsers": {
+ "type": [
+ "array",
+ "null"
+ ],
+ "items": {
+ "type": "string",
+ "format": "misskey:id"
+ }
+ },
"hiddenTags": {
"type": [
"array",
@@ -14367,6 +14401,15 @@
"$ref": "#/components/schemas/Error"
},
"examples": {
+ "FOLLOWED_USER_DUPLICATED": {
+ "value": {
+ "error": {
+ "message": "Some items in \"defaultFollowedUsers\" and \"forciblyFollowedUsers\" are duplicated.",
+ "code": "FOLLOWED_USER_DUPLICATED",
+ "id": "bcf088ec-fec5-42d0-8b9e-16d3b4797a4d"
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -20412,6 +20455,16 @@
}
}
},
+ "CANNOT_BLOCK_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot block that user due to server policy.",
+ "code": "CANNOT_BLOCK_DUE_TO_SERVER_POLICY",
+ "id": "e2f04d25-0d94-4ac3-a4d8-ba401062741b",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -36717,6 +36770,16 @@
}
}
},
+ "CANNOT_UNFOLLOW_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot unfollow that user due to server policy.",
+ "code": "CANNOT_UNFOLLOW_DUE_TO_SERVER_POLICY",
+ "id": "19f25f61-0141-4683-99dc-217a88d633cb",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -54331,6 +54394,16 @@
}
}
},
+ "CANNOT_MUTE_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot mute that user due to server policy.",
+ "code": "CANNOT_MUTE_DUE_TO_SERVER_POLICY",
+ "id": "15273a89-374d-49fa-8df6-8bb3feeea455",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -54868,6 +54941,16 @@
}
}
},
+ "CANNOT_MUTE_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot mute that user due to server policy.",
+ "code": "CANNOT_MUTE_DUE_TO_SERVER_POLICY",
+ "id": "15273a89-374d-49fa-8df6-8bb3feeea455",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": { |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14769 +/- ##
===========================================
- Coverage 41.74% 41.37% -0.38%
===========================================
Files 1549 1558 +9
Lines 196555 201779 +5224
Branches 2767 2592 -175
===========================================
+ Hits 82055 83477 +1422
- Misses 113939 117734 +3795
- Partials 561 568 +7 ☔ View full report in Codecov by Sentry. |
/preview |
フォロー解除できないユーザーのほうのプロパティ名が微妙なので募集中 |
permanentFollowedはpermientが形容詞だけどfollowedは名詞じゃないので品詞的におかしくてpermanentlyのほうが適切だとは思います。そもそも単語選択レベルでもっといいのあるかもだけど |
forceとか |
fixedとか |
の両方を取り入れて |
そうした |
ピン留めユーザーの設定はacct形式での入力を求めるのに対して、(処理を簡略化する都合上)こちらではユーザーIDを入力させるようにしてあるけど、改めたほうがいいかしら(管理者画面なので「IDを入力せよ」と明記するだけで良い説はある (すでにそれはやってある)) |
これユーザーにとって不便になる影響の方が大きいんじゃないかしら |
だからサーバーの運用に合わせて管理者がうまいこと使い分ければ良さそう(ユーザーでフォロー解除できる方も別個の設定として存在しているので) |
ほむん |
お知らせ管理画面周りみたく、別途MkInfoか何かで「フォロー解除できない方の設定は最低限のアカウントにすることをおすすめします」とかを書くといいのかもしれない |
インポート機能使えばミュートやブロック不可制限を回避できそうな予感 |
あーそっちは塞いでなかったわね |
これどうだろう |
塞いだ |
UserMutingService.muteとかに当該ロジックあった方が自然だし漏れが無くなるけどそうしなかった理由はあるかしら |
あーそっちに入れるほうがいいか(別の内部的な処理とかち合うとややこしそうかなと思ったので) |
This reverts commit d96b2b8.
What
Why
Fix #12451
Additional info (optional)
フォロー解除できるユーザーと解除できないユーザーのフィールドを分けずに、別途スイッチでフォロー解除できるようにするかどうかを選べるようにすることも考えたけど、両方のフィールドを使う運用も考えられた(推奨するユーザーも必須なユーザーもはじめからフォローしておいてもらうような運用)ので両方別で残している
Checklist