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.

Sign up for GitHub

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

Jump to bottom

fix: preg_replace_callback(): Empty regular expression #53

Merged
merged 14 commits into from
Jun 12, 2024
Merged

fix: preg_replace_callback(): Empty regular expression #53

merged 14 commits into from
Jun 12, 2024

Conversation

imorland
Copy link
Member

@imorland imorland commented Jun 12, 2024 •
edited
Loading

  • fix: addresses preg_replace_callback error when badwords list is manually imported to the db
  • add basic first tests to ensure that posts with badwords are marked for approval
  • perf: store the compiled censors in cache rather than db
  • chore: migrate from npm to yarn

@imorland imorland changed the title add tests fix: preg_replace_callback(): Empty regular expression Jun 12, 2024
@imorland imorland added the bug label Jun 12, 2024
@imorland imorland marked this pull request as ready for review June 12, 2024 08:06
@imorland imorland merged commit 1e32bbb into master Jun 12, 2024
28 checks passed
@imorland imorland deleted the im/tests branch June 12, 2024 09:25
@DavideIadeluca DavideIadeluca mentioned this pull request Aug 2, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DavideIadeluca DavideIadeluca DavideIadeluca approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@imorland @DavideIadeluca @StyleCIBot