relax slur filter #322

Merged
DashEightMate merged 0 commits from layla/lemmy-hexbear:feature/relax-slurfilter into main 1 month ago
layla commented 3 months ago

this will allow for example "refutation" and "nagging"

this will allow for example "refutation" and "nagging"
layla added 1 commit 3 months ago
4f14945197 relax slur filter
Poster
Owner

These changes look alright. I wonder if we could separate the character substitutions you removed into a separate token and put a {1,} at the end of ((a|er)?(s|z)?). Therefore things like nagg won't match unless it ends with something.

I wouldn't mind adding this although I don't know what my perms are regarding pushing to open prs. Let me know what you think about the changes.

These changes look alright. I wonder if we could separate the character substitutions you removed into a separate token and put a `{1,}` at the end of `((a|er)?(s|z)?)`. Therefore things like `nagg` won't match unless it ends with something. I wouldn't mind adding this although I don't know what my perms are regarding pushing to open prs. Let me know what you think about the changes.
layla added 1 commit 1 month ago
layla added 1 commit 1 month ago
Poster

Not entirely sure what you mean with those changes. If you can't push to this PR you could just include the SLUR_REGEX you're suggesting in a comment and I'll take a look. As long as it allows the words users have wanted then it's probably fine with me.

Not entirely sure what you mean with those changes. If you can't push to this PR you could just include the `SLUR_REGEX` you're suggesting in a comment and I'll take a look. As long as it allows the words users have wanted then it's probably fine with me.
DashEightMate merged commit caa2884cd5 into main 1 month ago
layla deleted branch feature/relax-slurfilter 1 month ago
The pull request has been merged as caa2884cd5.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.