relax slur filter #322

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

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

this will allow for example "refutation" and "nagging"
layla added 1 commit 1 year 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 year ago
layla added 1 commit 1 year ago
Poster
Owner

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 year ago
layla deleted branch feature/relax-slurfilter 1 year 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.