aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | | | | | | Update Silence's scheduler to the new APIGravatar MarkKoz2020-06-20-26/+6
| | | | | | | |
* | | | | | | | Update Reminders's scheduler to the new APIGravatar MarkKoz2020-06-20-14/+15
| | | | | | | |
* | | | | | | | Update Filtering's scheduler to the new APIGravatar MarkKoz2020-06-20-10/+6
| | | | | | | |
* | | | | | | | Scheduler: add a method to schedule at a specific datetimeGravatar MarkKoz2020-06-20-0/+13
| | | | | | | |
* | | | | | | | Scheduler: add a method to schedule with a delayGravatar MarkKoz2020-06-20-0/+19
| | | | | | | |
* | | | | | | | Scheduler: rename "task" param to "coroutine"Gravatar MarkKoz2020-06-20-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Naming it "task" is inaccurate because `create_task` accepts a coroutine rather than a Task. What it does is wrap the coroutine in a Task.
* | | | | | | | Scheduler: drop _task suffix from method namesGravatar MarkKoz2020-06-20-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's redundant. After all, this scheduler cannot schedule anything else.
* | | | | | | | Scheduler: remove ignore_missing paramGravatar MarkKoz2020-06-20-10/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ability to use the `in` operator makes this obsolete. Callers can check themselves if a task exists before they try to cancel it.
* | | | | | | | Scheduler: use pop instead of get when cancellingGravatar MarkKoz2020-06-19-7/+7
| | | | | | | |
* | | | | | | | Scheduler: add support for in operatorGravatar MarkKoz2020-06-19-0/+4
| | | | | | | |
* | | | | | | | Scheduler: name tasksGravatar MarkKoz2020-06-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes them easier to identify when debugging.
* | | | | | | | Scheduler: directly take the awaitable to scheduleGravatar MarkKoz2020-06-19-23/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a major change which simplifies the interface. It removes the need to implement an abstract method, which means the class can now be instantiated rather than subclassed.
* | | | | | | | Scheduler: use separate logger for each instanceGravatar MarkKoz2020-06-19-24/+17
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Each instance now requires a name to be specified, which will be used as the suffix of the logger's name. This removes the need to manually prepend every log message with the name.
* | | | | | | Merge pull request #1009 from python-discord/bug/mod/bot-2a/webhook-clydeGravatar kwzrd2020-06-20-8/+57
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix 400 when "clyde" is in any webhook username
| * \ \ \ \ \ \ Merge branch 'master' into bug/mod/bot-2a/webhook-clydeGravatar kwzrd2020-06-20-100/+285
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #1015 from python-discord/kwzrd/pipenv-html-scriptGravatar Mark2020-06-19-0/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Add Pipenv script for html coverage report
| * | | | | | | | Pipenv: add script for html coverage reportGravatar kwzrd2020-06-19-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Similarly to the `report` script, this removes the need to invoke coverage when generating the html report.
* | | | | | | | | Merge pull request #1003 from ↵Gravatar Dennis Pham2020-06-19-16/+47
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/filters/bot-58/removers-ignore-dms Ignore DMs in token and webhook removers
| * | | | | | | | Merge branch 'master' into bug/filters/bot-58/removers-ignore-dmsGravatar Dennis Pham2020-06-19-85/+238
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #892 from Akarys42/#364-offensive-msg-autodeletionGravatar Joseph Banks2020-06-17-9/+80
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | #364 offensive msg autodeletion
| * | | | | | | | | Update deletion scheduler to use latest watchlist configurationGravatar Joseph Banks2020-06-17-17/+5
| | | | | | | | | |
| * | | | | | | | | Merge branch 'master' into #364-offensive-msg-autodeletionGravatar Joseph Banks2020-06-17-2517/+7259
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge pull request #991 from crazygmr101/feature/cooldown-tagGravatar Mark2020-06-17-0/+20
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Create cooldown.md
| * | | | | | | | | | Update bot/resources/tags/customcooldown.mdGravatar Daniel Nash2020-06-15-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| * | | | | | | | | | Merge branch 'master' into feature/cooldown-tagGravatar Joseph Banks2020-06-12-76/+179
| |\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add scheme to URLGravatar Daniel Nash2020-06-12-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Update customcooldown.mdGravatar Daniel Nash2020-06-12-0/+2
| | | | | | | | | | | |
| * | | | | | | | | | | Make title more specificGravatar Daniel Nash2020-06-12-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| * | | | | | | | | | | Update bot/resources/tags/customcooldown.mdGravatar Daniel Nash2020-06-10-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joseph Banks <[email protected]>
| * | | | | | | | | | | change _cd to message_cooldown Gravatar Daniel Nash2020-06-10-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apply suggestions from code review Co-authored-by: Joseph Banks <[email protected]>
| * | | | | | | | | | | Move the not rate-limited message into elseGravatar Daniel Nash2020-06-10-1/+3
| | | | | | | | | | | |
| * | | | | | | | | | | Update example to not be in a cogGravatar Daniel Nash2020-06-10-12/+10
| | | | | | | | | | | |
| * | | | | | | | | | | Rename to customcooldown.mdGravatar Daniel Nash2020-06-10-0/+0
| | | | | | | | | | | |
| * | | | | | | | | | | Update cooldown.mdGravatar Daniel Nash2020-06-07-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Create cooldown.mdGravatar Daniel Nash2020-06-07-0/+22
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #1001 from python-discord/bug/mod/bot-68/ban-strips-noneGravatar Mark2020-06-17-22/+38
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Check infraction reason isn't None before shortening it
| * | | | | | | | | | | | Format parameters with a more consistent styleGravatar MarkKoz2020-06-16-19/+19
| | | | | | | | | | | | |
| * | | | | | | | | | | | Add optional type annotations to reason in pardon funcsGravatar MarkKoz2020-06-16-2/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | Revise inaccurate typehint for Optional reasonGravatar kosayoda2020-06-16-18/+31
| | | | | | | | | | | | |
| * | | | | | | | | | | | Fix check for bot actor in infractionsGravatar MarkKoz2020-06-14-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason None check should be nested to avoid affecting the else/elif statements that follow.
| * | | | | | | | | | | | Check infraction reason isn't None before shortening itGravatar MarkKoz2020-06-14-5/+8
| | |_|_|_|_|_|_|/ / / / | |/| | | | | | | | | |
* | | | | | | | | | | | Add LMGTFY to domain blacklistGravatar Joseph Banks2020-06-17-0/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #987 from python-discord/help_channel_rediscacheGravatar Kieran Siek2020-06-16-53/+98
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Add persistence to the help channel system
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into help_channel_rediscacheGravatar Kieran Siek2020-06-16-243/+585
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Revise inaccurate docstring in RedisCacheGravatar kosayoda2020-06-16-2/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | Refactor nested if-statementGravatar kosayoda2020-06-16-5/+4
| | | | | | | | | | | | |
| * | | | | | | | | | | | Help channels: revise inaccurate commentGravatar MarkKoz2020-06-08-1/+1
| | | | | | | | | | | | |
| * | | | | | | | | | | | Help channels: use cache to remove cooldowns or re-schedule themGravatar MarkKoz2020-06-08-7/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the cache is more efficient since it can check only the users it expects to have a cooldown rather than searching all guild members. Furthermore, re-scheduling the cooldowns ensures members experience the full duration of the cooldown. Previously, all cooldowns were removed, regardless of whether they were expired.
| * | | | | | | | | | | | Help channels: add a function to schedule cooldown expirationGravatar MarkKoz2020-06-08-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moving this code into a separate function reduces redundancy down the line. This will also get used to re-scheduled cooldowns after a restart.
| * | | | | | | | | | | | Help channels: add a function to get in use timeGravatar MarkKoz2020-06-08-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Future code will also need to get this time, so moving it out to a separate function reduces redundancy.