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

spec(backend/NoteCreateService): ローカルユーザーがまだ誰もフォローしていないリモートユーザーによる通知を引き起こす可能性のある投稿を拒否できるように (MisskeyIO#462) (patched) #149

Merged
merged 4 commits into from
Feb 19, 2024

Conversation

anatawa12
Copy link
Collaborator

@anatawa12 anatawa12 commented Feb 19, 2024

Cherry-picked from 738b4d6, 1b3adcc, 33cb507, a27af00, 5c6236b

What

Fixes #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
Copy link
Collaborator Author

これ有効無効envでいいの?という気持ちが少し

Copy link

github-actions bot commented Feb 19, 2024

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

差分はこちら

Get diff files from Workflow Page

@Sayamame-beans
Copy link

これ有効無効envでいいの?という気持ちが少し

同じ気持ちです

@anatawa12
Copy link
Collaborator Author

configにする?

@anatawa12 anatawa12 force-pushed the block-mentions-from-non-followed branch from ec6c2a3 to f93be69 Compare February 19, 2024 18:28
@anatawa12
Copy link
Collaborator Author

した

@Sayamame-beans
Copy link

とりあえずこれをそのまま採用するという前提に立てば、configかサーバー側の設定項目雑に生やすかのどちらかかなぁ(実質前者のみ?)という感じでしょうか

@anatawa12 anatawa12 changed the title spec(backend/NoteCreateService): ローカルユーザーがまだ誰もフォローしていないリモートユーザーによる通知を… spec(backend/NoteCreateService): ローカルユーザーがまだ誰もフォローしていないリモートユーザーによる通知を引き起こす可能性のある投稿を拒否できるように (MisskeyIO#462) (patched) Feb 19, 2024
@anatawa12
Copy link
Collaborator Author

metaに追加は面倒なのでconfigにしました。

@anatawa12
Copy link
Collaborator Author

(changelogのコンフリクト面倒なのでできればマージしたいのでreview急いでほしい)

@Sayamame-beans
Copy link

む…
じゃあ見ます

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Sayamame-beans <[email protected]>
@Sayamame-beans
Copy link

else Approve

@anatawa12 anatawa12 merged commit 017663a into niri-la:develop Feb 19, 2024
14 checks passed
@anatawa12 anatawa12 deleted the block-mentions-from-non-followed branch February 19, 2024 18:57
@Sayamame-beans
Copy link

additional patchがあるっぽい
MisskeyIO#466

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