aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | Improvements to the Wolfram cog.Gravatar Leon Sandøy2019-09-14-7/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds clearer error messages for all the errors produced by the Wolfram cog if the API key is missing in the config, or if the key isn't valid anymore. It also humanizes the timedelta returned in the error users get when they run out their cooldown. Instead of telling them they need to wait 84000 seconds, it will now tell them they need to wait 23 hours, 59 minutes ...
* | | Make 'post_infraction' catch the right exceptionGravatar Sebastiaan Zeeff2019-09-14-11/+17
| | | | | | | | | | | | | | | | | | | | | | | | The internal 'api' of our API client has changed: It raises a custom RespondeCodeError instead of an `aiohttp.ClientError` when an API was not successful. I updated this utility function to catch the right exception and added handling for unknown users by notifying the user of that problem directly.
* | | Add API response dict to ResponseCodeErrorGravatar Sebastiaan Zeeff2019-09-14-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ReponseCodeError held a reference to `aiohttp.ResonseObject` to make sure the response data was available. However, the response data is not actually included in the Response Object, but needs to be awaited. Unfortunately, the ResponseCodeError is usually inspected after the connection has been closed, triggering a ClientConnectionError when the data was retrieved. I solved this by adding the awaited reponse data directly to our custom exception by awaiting the response.json() before raising the exception.
* | | Add error handlers for more command exceptionsGravatar MarkKoz2019-09-13-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | MissingPermissions, CheckFailure, DisabledCommand, and CommandOnCooldown will now have a simple message logged. * Log BotMissingPermissions and remove sending missing permissions as a message
* | | Fix one-off error in the !clean commandGravatar Sebastiaan Zeeff2019-09-13-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/413 The message indexing phase of the `!clean` did not account for the presence of the invocation message, resulting in a one-off error in the amount of messages being indexed. Fixed it by adding one to the amount of messages we index from the message history. Closes #413
* | | Fix bugs and inconsistencies in moderation cogGravatar Sebastiaan Zeeff2019-09-13-154/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recent changes and updates to the moderation cog introduced some inconsistencies that were causing bugs or differences in behavior between very similar commands. I've remedied the problems by: - Consistently making sure we stop if a post_infraction API call fails; - Factoring out the check for active infractions to a utility function; - Updating commands that expected a pre-migration API response format. In addition, I've also added function annotations.
* | | Update bot cog with recent changes.Gravatar Sebastiaan Zeeff2019-09-12-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | The bot cog was not updated with recent changes to our community, so I've: - Updated the links in the about embed to GitHub; - Added help-6 and help-7 to the codeblock detection.
* | | Add help-6 and help-7 to constantsGravatar Sebastiaan Zeeff2019-09-12-0/+4
| | | | | | | | | | | | | | | | | | We never added channel IDs for the new help channels to the constants after adding them, so I'm adding them in.
* | | Fix AntiSpam incorrectly invoking tempmute.Gravatar Sebastiaan Zeeff2019-09-12-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/400 The AntiSpam punish method incorrectly invoked the tempmute command, as it provided an unconverted duration argument. Since direct invocation of commands bypasses converters, the conversion of the duration string to a datetime object is now done manually. Closes #400
* | | Fixed incorrect API request field in superstarifyGravatar Sebastiaan Zeeff2019-09-12-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/409 The superstarify cog specified an incorrect infraction type in the API request in the on_member_join event listener. I've fixed it by giving it the correct infraction type, 'superstar'. closes #409
* | | Fixed AntiSpam cog reload bugGravatar Sebastiaan Zeeff2019-09-12-6/+3
|/ / | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/411 The AntiSpam code suffered from a bug where the attribute self.muted_role was not defined after reloading the cog. The bug was caused by the cog setting the attribute in on_ready instead of directly in __init__. Fixed by setting the attribute in the __init__. Closes #411
* | Merge pull request #402 from python-discord/update-contribGravatar Chris Goes2019-09-03-155/+157
|\ \ | | | | | | Update contrib doc for allowing edits from maintainers
| * | Remove pydocstyle pinning now that flake8-docstrings is fixedGravatar sco12019-09-03-145/+146
| | |
| * | Use pipenv lint script for pre-commit hookGravatar S. Co12019-09-03-10/+8
| | |
| * | Update contrib doc for allowing edits from maintainersGravatar S. Co12019-09-03-0/+3
|/ /
* | Merge pull request #399 from python-discord/django-updating-bot-logs-urlGravatar Leon Sandøy2019-08-28-1/+2
|\ \ | | | | | | Updating the deleted message front-end URL to the staff subdomain
| * | Changing deleted messages log to point at staff-subdomainGravatar Sebastiaan Zeeff2019-08-23-1/+2
| | |
* | | Do not send log messages below DEBUG to the site.Gravatar Johannes Christ2019-08-26-6/+14
|/ /
* | Merge pull request #397 from python-discord/django-role-api-changesGravatar Leon Sandøy2019-08-23-77/+143
|\ \ | | | | | | Django role api changes
| * | Changing logging interpolation style to f-stringsGravatar Sebastiaan Zeeff2019-08-23-4/+3
| | |
| * | Incorporating Mark's feedback on return annotationsGravatar Sebastiaan Zeeff2019-08-16-11/+15
| | |
| * | Kaizen: Change all string concats to f-stringsGravatar Sebastiaan Zeeff2019-08-16-7/+7
| | |
| * | Adding missing function annotationsGravatar Sebastiaan Zeeff2019-08-16-14/+14
| | |
| * | Updating role sync tests for position and adding tests for deletion detectionGravatar Sebastiaan Zeeff2019-08-15-21/+60
| | |
| * | Adding support for storing role positions and deleting rolesGravatar Sebastiaan Zeeff2019-08-15-27/+51
|/ /
* | Merge pull request #395 from kosayoda/no_fun_allowedGravatar Chris Goes2019-08-14-54/+0
|\ \ | | | | | | Remove fun cog
| * | Remove fun cogGravatar kosayoda2019-08-14-54/+0
| | |
* | | Merge pull request #396 from python-discord/logging-handlerGravatar Johannes Christ2019-08-14-2/+80
|\ \ \ | | | | | | | | Write logs into the site
| * | | Add a site logging handler.Gravatar Johannes Christ2019-08-14-2/+80
|/ / /
* | | Merge pull request #392 from python-discord/django-readd-eventsGravatar Mark2019-08-05-20/+125
|\ \ \ | | | | | | | | Re-add command error event handler
| * | | Blame Mark.Gravatar Johannes Christ2019-08-05-1/+1
| | | |
| * | | Be more helpful.Gravatar Johannes Christ2019-08-05-0/+1
| | | |
| * | | Fix typo in bot/cogs/error_handler.py. Gravatar Johannes Christ2019-08-05-1/+1
| | | | | | | | | | | | | | | | | | | | Originally authored by @MarkKoz. Co-Authored-By: Mark <[email protected]>
| * | | Suppress response code errors on tag reinvoke.Gravatar Johannes Christ2019-08-05-1/+3
| | | |
| * | | Finalize error handling.Gravatar Johannes Christ2019-08-05-12/+15
| | | |
| * | | Update code to make use of the new `ResponseCodeError`.Gravatar Johannes Christ2019-08-05-18/+22
| | | |
| * | | Raise specific exception for non-200s.Gravatar Johannes Christ2019-08-04-9/+21
| | | |
| * | | Handle more API status codes.Gravatar Johannes Christ2019-07-28-1/+5
| | | |
| * | | Move error handling to more descriptive `ErrorHandler` cog.Gravatar Johannes Christ2019-07-28-18/+14
| | | |
| * | | Drop user updating from `events` cog.Gravatar Johannes Christ2019-07-28-203/+0
| | | |
| * | | Revert 4d35f8f7137edb97e1124fa9087bd86399398047.Gravatar Johannes Christ2019-07-28-0/+286
|/ / /
* | | Merge pull request #386 from python-discord/django-mergeGravatar Johannes Christ2019-07-21-2766/+1635
|\ \ \ | | | | | | | | Django Merge
| * | | Remove unused importGravatar MarkKoz2019-07-16-1/+0
| | | |
| * | | Merge remote-tracking branch 'origin/master' into django-mergeGravatar MarkKoz2019-07-16-13/+11
| |\| | | | | | | | | | | | | | Merge user-event whitelists and special character support for off topic names.
| | * | Whitelisting the kivy discord server from our filtersGravatar Leon Sandøy2019-07-06-0/+1
| | |/
| | * Update off_topic_names.pyGravatar Jeremiah Boby2019-06-23-2/+2
| | |
| | * Merge pull request #373 from python-discord/off-topic-convertGravatar Jeremiah Boby2019-06-23-6/+6
| | |\ | | | | | | | | Accept uppercase and punctuation for off-topic names
| | | * Merge branch 'master' into off-topic-convertGravatar Leon Sandøy2019-06-22-7/+5
| | | |\ | | | |/ | | |/|
| | * | Anti-spam should use the same whitelist as FiltersGravatar Joseph Banks2019-06-22-7/+2
| | | |
| | * | add user events announcements to the constants.py classGravatar Joseph Banks2019-06-22-0/+1
| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-29 22:17:02 +0000