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

Chat Filter #6901

Merged
merged 9 commits into from
Aug 22, 2024
Merged

Chat Filter #6901

merged 9 commits into from
Aug 22, 2024

Conversation

Git-Nivrak
Copy link
Contributor

@Git-Nivrak Git-Nivrak commented Aug 8, 2024

About the pull request

Adds a chat filter which automatically prevents messages in-character, OOC, LOOC or dchat from being sent if they contain certain words, this will notify the user and warn them that they will be banned should they try to circumvent it.

This is obviously just a config option so if this is merged the config will have to be edited accordingly.

Explain why it's good for the game

Instead of instantly banning people for saying certain words, simply prevents them from saying them in the first place - this way no harm is done to anyone who might be offended as well as the player which (possibly mistakenly) said the word not being banned and instead let know the word is not allowed.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
config: Added config options for filtering out certain words
/:cl:

@github-actions github-actions bot added the Config drag and drop label Aug 8, 2024
@BeagleGaming1
Copy link
Contributor

Is this filter using regex? If not, it should

code/modules/mob/living/say.dm Outdated Show resolved Hide resolved
code/modules/mob/living/say.dm Outdated Show resolved Hide resolved
config/example/word_filter.txt Outdated Show resolved Hide resolved
config/example/config.txt Outdated Show resolved Hide resolved
@harryob harryob enabled auto-merge August 22, 2024 13:13
@harryob harryob added this pull request to the merge queue Aug 22, 2024
@harryob harryob removed this pull request from the merge queue due to a manual request Aug 22, 2024
@harryob harryob enabled auto-merge August 22, 2024 13:37
@harryob harryob added this pull request to the merge queue Aug 22, 2024
Merged via the queue into cmss13-devs:master with commit d628fc6 Aug 22, 2024
27 checks passed
cm13-github added a commit that referenced this pull request Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Config drag and drop
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants