ローカルユーザーが誰もフォローしていないリモートユーザーによるノートのブロックの改善

[anatawa12]

What

Fixes some portion of #146

Why

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

[anatawa12]

どちらの設定もコンパネからいじれます

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

--- base
+++ head
@@ -743,6 +743,15 @@
                         "string",
                         "null"
                       ]
+                    },
+                    "nirilaBlockMentionsFromUnfamiliarRemoteUsers": {
+                      "type": "boolean"
+                    },
+                    "nirilaAllowedUnfamiliarRemoteUserIds": {
+                      "type": "array",
+                      "items": {
+                        "type": "string"
+                      }
                     }
                   },
                   "required": [
@@ -850,7 +859,9 @@
                     "urlPreviewMaximumContentLength",
                     "urlPreviewRequireContentLength",
                     "urlPreviewUserAgent",
-                    "urlPreviewSummaryProxyUrl"
+                    "urlPreviewSummaryProxyUrl",
+                    "nirilaBlockMentionsFromUnfamiliarRemoteUsers",
+                    "nirilaAllowedUnfamiliarRemoteUserIds"
                   ]
                 }
               }
@@ -13353,6 +13364,15 @@
                       "string",
                       "null"
                     ]
+                  },
+                  "nirilaBlockMentionsFromUnfamiliarRemoteUsers": {
+                    "type": "boolean"
+                  },
+                  "nirilaAllowedUnfamiliarRemoteUserIds": {
+                    "type": "array",
+                    "items": {
+                      "type": "string"
+                    }
                   }
                 }
               }

Get diff files from Workflow Page

[anatawa12]

lintは後でやります

[Sayamame-beans]

流れてきたノートの通知対象ユーザーの中に、1人でも通知を許可するユーザーが居たら、そのノートは(対象者全員に)通知されるって感じであってます?
雰囲気的には大体良さそうですかね～

[anatawa12]

流れてきたノートの通知対象ユーザーの中に、1人でも通知を許可するユーザーが居たら、そのノートは(対象者全員に)通知されるって感じであってます?

No 全員が許可されてないとだめです。(じゃないとスパムの抜け道になるので)

[Sayamame-beans]

流れてきたノートの通知対象ユーザーの中に、1人でも通知を許可するユーザーが居たら、そのノートは(対象者全員に)通知されるって感じであってます?

No 全員が許可されてないとだめです。(じゃないとスパムの抜け道になるので)

early returnしててそういうロジックになってないような気が…?

[anatawa12]

入ってないときにreturn faldeしたつもりになってた

[Sayamame-beans]

他はなんとなく良さそう

[Sayamame-beans]

ok