-
-
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: 過去のノートを非公開化/フォロワーのみ表示可能にできる機能 #14814
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14814 +/- ##
============================================
+ Coverage 19.45% 41.24% +21.78%
============================================
Files 722 1559 +837
Lines 102566 201838 +99272
Branches 991 3680 +2689
============================================
+ Hits 19958 83254 +63296
- Misses 82058 117980 +35922
- Partials 550 604 +54 ☔ View full report in Codecov by Sentry. |
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -50278,6 +50278,18 @@
"requireSigninToViewContents": {
"type": "boolean"
},
+ "makeNotesFollowersOnlyBefore": {
+ "type": [
+ "integer",
+ "null"
+ ]
+ },
+ "makeNotesHiddenBefore": {
+ "type": [
+ "integer",
+ "null"
+ ]
+ },
"isBot": {
"type": "boolean"
},
@@ -77156,6 +77168,18 @@
"requireSigninToViewContents": {
"type": "boolean"
},
+ "makeNotesFollowersOnlyBefore": {
+ "type": [
+ "number",
+ "null"
+ ]
+ },
+ "makeNotesHiddenBefore": {
+ "type": [
+ "number",
+ "null"
+ ]
+ },
"instance": {
"type": "object",
"properties": { |
ピン留めされたノートは除外したいけどパフォーマンス的に困難な可能性がある |
いやそうでもないな |
いやそうかも |
最低限、TLに流れなくてもユーザーページからは見られたら良いかもしれませんね(ユーザー情報にピン留めノートの情報含まれてませんでしたっけ) |
ユーザーページからは見れるようにするのは多分簡単に実装できるけど、投稿詳細に遷移したら非公開扱いになるのが混乱を招きそう |
ピン留めノートの情報はUser自体は持っていないから、投稿詳細などでもピン留め考慮する場合はいちいち追加でクエリ走らせる必要が出てくる |
まあ見れないよりはマシかしら |
それは確かに… |
私もそう思ったので、"最低限"としての提案でした |
もしくはUserにも非正規化としてピン留め情報持たせるようにするとか |
IDだけでも持っていれば照らし合わせることは出来そう… |
別のPRでやるか |
レビュワーを募集しています |
※Chromaticはプラン上限を超過したため確認不能 |
マージするか |
🙏🏻 |
* ap: `_misskey_makeNotes*Before` misskey-dev/misskey#14814 * fix ambiguous description
What
Resolve #14802
Why
Additional info (optional)
Checklist