aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | | * | 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]>
| | | | | | | * Removed `zen` tag due `!zen` command exist.Gravatar Karlis S2020-03-23-20/+0
| |_|_|_|_|_|/ |/| | | | | |
* | | | | | | Merge pull request #830 from ks129/bug-fixesGravatar Mark2020-03-16-1/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | ModLog Channel Update formatting fix.
| * \ \ \ \ \ \ Merge branch 'master' into bug-fixesGravatar Mark2020-03-16-0/+80
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #828 from python-discord/feat/test/765/command-shadowingGravatar Mark2020-03-16-0/+80
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Test for command name and alias shadowing
| * \ \ \ \ \ \ \ Merge branch 'master' into feat/test/765/command-shadowingGravatar Mark2020-03-16-106/+937
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Cog tests: comment some code for clarificationGravatar MarkKoz2020-03-16-0/+3
| | | | | | | | |
| * | | | | | | | Cog tests: fix error on import due to discord.ext.tasks.loopGravatar MarkKoz2020-03-13-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tasks extensions loop requires an event loop to exist. To work around this, it's been mocked.
| * | | | | | | | Cog tests: fix leading space in aliases without parentsGravatar MarkKoz2020-03-13-1/+1
| | | | | | | | |
| * | | | | | | | Cog tests: add a test for duplicate command names & aliasesGravatar MarkKoz2020-03-13-0/+17
| | | | | | | | |
| * | | | | | | | Cog tests: fix duplicate commands being yieldedGravatar MarkKoz2020-03-13-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | discord.py yields duplicate Command objects for each alias a command has, so the duplicates need to be removed on our end.
| * | | | | | | | Cog tests: fix nested modules not being foundGravatar MarkKoz2020-03-13-9/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rename `walk_extensions` to `walk_modules` because some extensions don't consist of a single module
| * | | | | | | | Cog tests: fix duplicate cogs being yieldedGravatar MarkKoz2020-03-13-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Have to check the modules are equal to prevent yielding imported cogs.
| * | | | | | | | Cog tests: fix Cog type check in `walk_cogs`Gravatar MarkKoz2020-03-13-3/+3
| | | | | | | | |
| * | | | | | | | Cog tests: add a function to yield all commandsGravatar MarkKoz2020-03-13-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will help reduce nesting in the actual test.
| * | | | | | | | Cog tests: add a function to get all qualified names for a cmdGravatar MarkKoz2020-03-13-0/+8
| | | | | | | | |
| * | | | | | | | Cog tests: add a function to get all cogsGravatar MarkKoz2020-03-13-0/+7
| | | | | | | | |
| * | | | | | | | Cog tests: add a function to get all extensionsGravatar MarkKoz2020-03-13-0/+11
| | | | | | | | |
| * | | | | | | | Cog tests: add a function to get all commandsGravatar MarkKoz2020-03-09-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For tests, ideally creating instances of cogs should be avoided to avoid extra code execution. This function was copied over from discord.py because their function is not a static method, though it still works as one. It was probably just a design decision on their part to not make it static.
| * | | | | | | | Cog tests: create boilerplate for command name testsGravatar MarkKoz2020-03-09-0/+7
| | |_|_|_|_|/ / | |/| | | | | |
| | * | | | | | Update explanation comment so it explains what happensGravatar S. Co12020-03-16-2/+3
| | | | | | | |
| | * | | | | | (Mod Log): Added comment about channel update formatting change.Gravatar Karlis S2020-03-16-0/+2
| | | | | | | |
| | * | | | | | (Mod Log): Fixed case when `on_guild_channel_update` old or new value is ↵Gravatar ks1232020-03-16-1/+1
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | empty and with this message formatting go wrong.
* | | | | | | Merge pull request #803 from RohanJnr/tags_overhaulGravatar Shirayuki Nekomata2020-03-16-106/+937
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Removed API dependency for Tags
| * | | | | | Fixed tag search via contents, any keywords.Gravatar Shirayuki Nekomata2020-03-16-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed `!tag search any` raises `AttributeError`. Changed default value of `keywords` from `None` to `'any'`. This will make it search for keyword `'any'` when there is no keyword.
| * | | | | | not awaiting _get_tags_via_content() method as it is non-asyncGravatar RohanJnr2020-03-15-2/+2
| | | | | | |
| * | | | | | Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵Gravatar RohanJnr2020-03-15-9/+15
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | * | | | | | Tags: add restrictions 1 & 9 from YouTube ToS to ytdl tagGravatar MarkKoz2020-03-14-2/+6
| | | | | | | |
| | * | | | | | Tags: use constant for command prefix in embed footerGravatar MarkKoz2020-03-14-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add a constant for the footer text * Import constants module rather than its classes
| * | | | | | | convert _get_tags_via_content() method to non-asyncGravatar RohanJnr2020-03-15-1/+1
| |/ / / / / /
| * | | | | | Remove line that calls get_tags() methodGravatar RohanJnr2020-03-14-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tags have now been shifted from the database to being static files and hence the get_tags() method has undergone changes. It now dosen't fetch from the database but looks at the local files and we need not call it more than once.
| * | | | | | Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵Gravatar RohanJnr2020-03-12-22/+200
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | * \ \ \ \ \ Merge branch 'master' into tags_overhaulGravatar Rohan2020-03-12-22/+200
| | |\ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
* | | | | | | | 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.