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

スパム軽減用フィルタに関する修正 #1

Merged
merged 4 commits into from
Feb 25, 2024

Conversation

Sayamame-beans
Copy link

@Sayamame-beans Sayamame-beans commented Feb 23, 2024

What

Why

  • IOパッチの遮断条件を修正するパッチを当てるため (Enhance: ローカルユーザーがまだ誰もフォローしていないリモートユーザーによる投稿のフィルターの条件の調整)
  • Fix: 禁止ワード設定によってリモートの投稿がブロックされると、InboxキューにDelayed Jobとして残ってしまう問題を修正
  • Fix: ローカルユーザーがまだ誰もフォローしていないリモートユーザーによる、通知を引き起こす可能性のある投稿を拒否すると、InboxキューにDelayed Jobとして残ってしまう問題を修正

Additional info (optional)

IOパッチを当てた際にChangelogが追加されていないようだったので、こちらのPRでも追加していません。

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

u1-liquid and others added 4 commits February 23, 2024 21:39
* refactor: use IdentifiableError instead of NoteCreateService.ContainsProhibitedWordsError

* fix: notes with prohibited words are reprocessed with delay

* docs(changelog): 禁止キーワードを含むノートがDelayed Queueに追加されて再処理される問題

* lint: fix lint errors

* fix: rethrowするべきなのにrethrowし忘れていたのを修正
@anatawa12
Copy link

note: storybookが落ちてるのはchromaticのトークンがないためなので問題があるとは限らないです

Copy link
Owner

@i544c-me i544c-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Sayamame-beans @anatawa12
LGTM です!修正のプルリクをありがとうございます 🙏

一箇所だけ気になったのでコメントしました!

Copy link
Owner

@i544c-me i544c-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

疑問が解消できたのでこれでマージします!ありがとうございます🙌

@i544c-me i544c-me merged commit b2c6e60 into i544c-me:custom Feb 25, 2024
1 of 2 checks passed
@Sayamame-beans Sayamame-beans deleted the filter-patch branch February 25, 2024 09:34
i544c-me pushed a commit that referenced this pull request Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants