hexbear-collective
/
lemmy-hexbear
31
14
Fork 7
Code Issues 37 Pull Requests 3 Projects Releases Wiki Activity
Labels Milestones
New Pull Request

relax slur filter #322

Merged
DashEightMate merged 0 commits from layla/lemmy-hexbear:feature/relax-slurfilter into main 2 years ago
Conversation 2 Commits N/A Files Changed N/A
layla commented 2 years ago
Owner

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

this will allow for example "refutation" and "nagging"
layla added 1 commit 2 years ago
4f14945197 relax slur filter
DashEightMate commented 2 years ago
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 2 years ago
layla added 1 commit 2 years ago
layla commented 2 years 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 2 years ago
layla deleted branch feature/relax-slurfilter 2 years ago
The pull request has been merged as caa2884cd5.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
Back End bug Can't Fix db Doing Feature feedback Front End Legal lemmy-reported Mobile moderation research-needed Security To Do triage UX waiting-on-upstream Won't Fix
bug
Something is not working design pending
In the UX / UI design stage. Not ready for software engineering comrades. documentation
Requires changes to docs dual sided
Requires work on both the backend and frontend duplicate
This issue or pull request already exists E-easy
Relatively easy to fix/implement E-hard
Difficult to fix or implement E-medium
Somewhat difficult to implement good first issue
Issues to get started with help wanted
Need some help high priority
items at the top of our "to do" list invalid
Something is wrong new feature
New feature / enhancement nice to have
A fun feature that isn't urgent question
More information is needed scoping
Not ready for development, still in brainstorm / scoping stage wontfix
This won't be fixed
No Label Back End bug Can't Fix db Doing Feature feedback Front End Legal lemmy-reported Mobile moderation research-needed Security To Do triage UX waiting-on-upstream Won't Fix bug design pending documentation dual sided duplicate E-easy E-hard E-medium good first issue help wanted high priority invalid new feature nice to have question scoping wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'layla/lemmy-hexbear:feature/relax-slurfilter'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes