-
-
Notifications
You must be signed in to change notification settings - Fork 715
Origin/feat/word scanner #5 #3287
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
Closed
AntonYder
wants to merge
19
commits into
python-discord:main
from
SEF-Group-25:origin/feat/word-scanner-#5
Closed
Origin/feat/word scanner #5 #3287
AntonYder
wants to merge
19
commits into
python-discord:main
from
SEF-Group-25:origin/feat/word-scanner-#5
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
feat: Added untrack command to stop tracking a specific word #8
This checks if a user triggers the track function too many times maliciously, since discord bot can already filter messages with spam. This adds test cases for the feature according to the requirements, and implement the feature.
feat: Add word tracking functionality with track and tracked commands #4
Feat/9 add json
Feat/4 add command
Waiting for helper functions and is assuming the existance of "channel_word_trackers" list. Will need to make more imports once they have been created.
The bot uses cogs that need to be loaded and requires that the function is under a class
Added init file and comments with content for testing. Forgot to add self so added it and finaly prepared for some helper functions which changed name or now have import
feat: Implemented tests for WordTracker.py #14
Was expecting int when it was in fact a string. Also prepped for send DM to be merged after detect is pushed to main
Issues when the bot tries to load the spam_check as a cog which stops the detect.py cog from propperly loading
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Integrates all other features except DMS into the detect.
Might be an issue where the detect_spam has to be under utls and not stalking_system though due to cog issues.