aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | Merge branch 'master' into tags_overhaulGravatar Rohan2020-02-29-895/+2636
| | |\ \ \ \
| * | | | | | Caching all the tags when the bot has loaded(caching only once) insted of ↵Gravatar RohanJnr2020-02-29-21/+18
| |/ / / / / | | | | | | | | | | | | | | | | | | caching it after the tags command is used.
| * | | | | Re-corrected the lines which I had changed by mistakeGravatar RohanJnr2020-02-29-1/+2
| | | | | |
| * | | | | added white spaces on statements before bullet points for proper rendering ↵Gravatar RohanJnr2020-02-29-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | of points on github
| * | | | | Added all the tag files in resources and modified cogs/tags.py file to ↵Gravatar RohanJnr2020-02-29-84/+919
| | | | | | | | | | | | | | | | | | | | | | | | access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
| | | | * | Escape markdown in voice updatesGravatar Jeremiah Boby2020-03-11-7/+14
| | | | | |
| | | | * | Escape markdown in member updatesGravatar Jeremiah Boby2020-03-11-1/+3
| | | | | |
| | | | * | Escape markdown in watchlist triggersGravatar Jeremiah Boby2020-03-11-2/+3
| | | | | |
| | | | * | Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2020-03-11-965/+903
| | | | |\ \ | |_|_|_|/ / |/| | | | |
| | | | * | Merge branch 'master' into spoiler-checkGravatar Leon Sandøy2020-02-29-297/+857
| | | | |\ \
| | | | * \ \ Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2020-02-27-2042/+5374
| | | | |\ \ \
| | | | * \ \ \ Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2019-11-28-2694/+5095
| | | | |\ \ \ \
| | | | * | | | | Expand spoilers to match multiple interpretationsGravatar Jeremiah Boby2019-11-28-1/+11
| | | | | | | | |
| | | | * | | | | Update spoiler regex to support multi-line spoilersGravatar Jeremiah Boby2019-10-21-1/+1
| | | | | | | | |
| | | | * | | | | Check partially hidden words against the wordlistGravatar Jeremiah Boby2019-10-13-1/+2
| | | | | | | | |
| | | | | | | | * Updated Docstrings, parameters, and log messagesGravatar Daniel Brown2020-03-20-13/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Docstrings for `apply_ban()` have been edited to mention that the method also removes a banned user from the watch list. - Parameter `banned` in `apply_unwatch()` was changed to `send_message` in order to be more general. Boolean logic was swapped to coincide with that change. - `apply_unwatch()`'s sent message moved to the bottom of the method for clarity. Added `return`s to the method to exit early if no message needs to be sent. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | | | * Updated doc strings to be more descriptiveGravatar Daniel Brown2020-03-20-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | | | * Nesting reduced, logging cleaned up and made clearerGravatar Daniel Brown2020-03-16-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | | | * Update bot/cogs/moderation/infractions.pyGravatar Daniel Brown2020-03-14-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | | | * Verification Cog Kaizen ChangesGravatar Daniel Brown2020-03-13-35/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Kaizen: - Cut down on the size of the import line by changing the imports from bot.constants to instead just importing the constants. This will help clarify where certain constants are coming from. - The periodic checkpoint message will no longer ping `@everyone` or `@Admins` when the bot detects that it is being ran in a debug environment. Message is now a simple confirmation that the periodic ping method successfully ran. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | | | * Corrected expiration check logic and cog loadingGravatar Daniel Brown2020-03-12-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bugs fixed: - Previously, the code would check to see if `'expires_at'` was in the kwargs, which after testing I came to find out that it is regardless of the duration of the ban. It has sense been changed to use a `.get()` in order to do a proper comparison. - Code previously attempted to load from the `"BigBrother"` cog which is the incorrect spelling. Changed it to `"Big Brother"` to correct this. Logging Added: - Additional trace logs added to both the `infractions.py` file as well as `bigbrother.py` to assist with future debugging or testing. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | | | * Merge remote-tracking branch 'origin/hemlock-perma-ban-watch-removal' into ↵Gravatar Daniel Brown2020-03-10-0/+0
| | | | | | | | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hemlock-perma-ban-watch-removal
| | | | | | | | | * Added BigBrother Helper MethodsGravatar Daniel Brown2020-03-06-7/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added apply_unwatch() and migrated the code from the unwatch command to it. This will give us more control regarding testing and also determining when unwatches trigger. - Added apply_watch() and migrated the code from the watch command to it. Again, this will assist with testing and could make it easier to automate adding to the watch list if need be. - Added unwatch call to apply_ban. User will only be removed from the watch list if they were permanently banned. They will not be removed if it was only temporary. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | | | * | Added BigBrother Helper MethodsGravatar Daniel Brown2020-03-09-7/+28
| |_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added apply_unwatch() and migrated the code from the unwatch command to it. This will give us more control regarding testing and also determining when unwatches trigger. - Added apply_watch() and migrated the code from the watch command to it. Again, this will assist with testing and could make it easier to automate adding to the watch list if need be. - Added unwatch call to apply_ban. User will only be removed from the watch list if they were permanently banned. They will not be removed if it was only temporary. Signed-off-by: Daniel Brown <[email protected]>
* | | | | | | | | Merge pull request #822 from python-discord/bug/mod/792/null-attachmentsGravatar Eivind Teig2020-03-09-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | ModLog: fix posting null attachments for deleted message logs
| * \ \ \ \ \ \ \ \ Merge branch 'master' into bug/mod/792/null-attachmentsGravatar Eivind Teig2020-03-09-6/+103
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge pull request #825 from python-discord/token-detection-fixGravatar Eivind Teig2020-03-09-3/+10
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Refactor token detection to check all potential substrings in message
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into token-detection-fixGravatar Eivind Teig2020-03-09-3/+93
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #826 from python-discord/voteGravatar Eivind Teig2020-03-09-0/+21
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Implement vote command.
| * | | | | | | | | | | Disambiguate codepoint value.Gravatar scragly2020-03-09-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The usage of 127462 as a unicode start point isn't super clear for other devs coming across the code in future, so assigning it to a nicely named variable with an accompanying inline comment should help make things clearer.
| * | | | | | | | | | | Check lower bound for vote options.Gravatar scragly2020-03-09-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the vote command receives less than 2 options, it's not being used for it's intended usage and is considered a user input error.
| * | | | | | | | | | | Merge branch 'master' into voteGravatar scragly2020-03-09-2/+2
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #824 from python-discord/antimalware-fixGravatar Mark2020-03-08-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Fix filtered extension string out of scope for antismalware log message
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into antimalware-fixGravatar Mark2020-03-08-1/+70
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Fix filtered extension string out of scope for log messageGravatar S. Co12020-03-08-2/+2
| | |_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | * Fix typo in file extensions list comprehension
| | * | | | | | | | | | Implement vote command.Gravatar scragly2020-03-09-0/+18
| |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The vote command takes a given list of options and generates a simple message and corresponding reactions for each so members can quickly take a vote on a subject during in-server discussions and meetings.
* | | | | | | | | | | Merge pull request #823 from ↵Gravatar Mark2020-03-08-1/+70
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/tag-search-searches-tags-via-content-instead-of-names Subcommand `search` and `search any` to search inside tags' contents and not names.
| * | | | | | | | | | Refactored to use paginator like normal `!tag`Gravatar Shirayuki Nekomata2020-03-09-12/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Split `_get_tags_via_content` - introduce `_send_matching_tags` - `_send_matching_tags` will send and paginate like `!tag` - Simplified `is_plural` even more.
| * | | | | | | | | | Fixed `is_plural` counting 1 less space.Gravatar Shirayuki Nekomata2020-03-08-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | | | | | | | | | Refactored dense codes, removed obvious type hint.Gravatar Shirayuki Nekomata2020-03-08-15/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Show the process of sanitizing the List[str] `keywords_processed`. - Show the process of finding tag for `matching_tags` ( was `founds` ). - Refactored the logic to find boolean `is_plural`. - Minor wording changes for docstring.
| * | | | | | | | | | Fixed searching for `,` returing all tags. Made it more descriptive when ↵Gravatar Shirayuki Nekomata2020-03-08-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | multiple tags are found. - Added a truthy check for each `query` since `','.split()` returns a list of two empty strings. - Changed from `Did you mean ...` to `Here are the tags containing the given keyword(s):` to be much more descriptive about the results - they are `tag` and not `term` to be searched.
| * | | | | | | | | | Refactored tag searching via keywords in contentsGravatar Shirayuki Nekomata2020-03-08-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Refactored `if` block - change to only send result when there is any result. - Added better type hinting for `check` argument of `_get_tags_via_content` - changed from `callable` to `Callable[[Iterable], bool]`. Thanks to @markkoz 's reviews Co-Authored-By: Mark <[email protected]>
| * | | | | | | | | | Implemented `search` as a subcommand for `tag` that will search in contents ↵Gravatar Shirayuki Nekomata2020-03-08-1/+50
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instead of names - `!tag search` will search for multiple keywords, separated by comma, and return tags that has ALL of these keywords. ` !tag search any` is the same as `!tag search` but it return tags that has ANY of the keyword instead.
| * / / / / / / / / Refactor token detection to check all potential substrings in messageGravatar S. Co12020-03-08-3/+10
|/ / / / / / / / /
| * / / / / / / / ModLog: fix posting null attachments for deleted message logsGravatar MarkKoz2020-03-07-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If attachments are not given to `upload_log`, an empty list is used. By default, `zip_longest` uses `None` ass the fill value, so each message was getting paired with a `None` (AKA null) attachment. The filed in the DB is non-nullable so an empty list must be used instead. Fixes #792
* | | | | | | | Merge pull request #820 from python-discord/feat/ci/pre-commit-tweaksGravatar Eivind Teig2020-03-06-2/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Pre-commit tweaks
| * | | | | | | | Remove excludes from pre-commitGravatar MarkKoz2020-03-05-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was excluding files that are already ignored by git. Pre-commit respects git ignore, even with --all-files, so these ignores were redundant.
| * | | | | | | | CI: don't show diff after pre-commit hooksGravatar MarkKoz2020-03-05-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's noisy, messy output. It's not of much benefit anyway as users can run git diff locally if they really need to see a diff. They have to do work locally anyway since CI won't commit the fixes pre-commit makes.
* | | | | | | | Merge pull request #643 from python-discord/zen-commandGravatar scragly2020-03-05-2/+86
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | implement !zen command.
| * \ \ \ \ \ \ \ Merge branch 'master' into zen-commandGravatar scragly2020-03-05-11/+9
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-31 08:27:08 +0000