aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | Moderation: explicitly set active value when posting infractionsGravatar MarkKoz2019-12-19-5/+5
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Required due to changes in the API making the active field required. See python-discord/site/pull/317
* | | | | | | | | | | Merge pull request #714 from python-discord/feat/watch/b713/show-name-if-watchedGravatar Johannes Christ2020-01-12-2/+2
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Watchchannels: show username in response for already watched users
| * | | | | | | | | | | Watchchannels: show username in response for already watched usersGravatar MarkKoz2019-12-30-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes it clear which user it is in the case of an ID being used to invoke the command.
* | | | | | | | | | | | Merge pull request #721 from python-discord/pep-empty-field-fixGravatar Johannes Christ2020-01-12-8/+6
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Add handling for empty PEP metadata values
| * | | | | | | | | | | | Add handling for empty PEP metadata valuesGravatar S. Co12020-01-11-8/+6
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some PEPs have a metadata field that is present but has an empty value (e.g. PEP 249), causing an exception to be raised when attempting to add it as an embed field value, which cannot be empty. This refactors the information parsing to prevent the field from being added if there is no value to provide, as well as cut down on copy+paste when populating fields in the embed.
* | | | | | | | | | | | Update Code Jam Participant IDGravatar Leon Sandøy2020-01-11-1/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #719 from python-discord/LordHemlock-Invite_White_ListGravatar Mark2020-01-09-0/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | Adding PyGame and Ren'Py to the invite white list
| * | | | | | | | | | | Adding PyGame and Ren'Py to the invite white listGravatar Daniel Brown2020-01-09-0/+2
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | With the addition of the #game-development channel, I would expect we'll see an increase in people needing specific help with the various engines and libraries that Python has to offer. As part of this, I'm adding the servers for PyGame and Ren'Py to the white list to help people get to the proper resources they might need.
| | * | | | | | | | | Move comment into if bodyGravatar Numerlor2020-01-01-2/+2
| | | | | | | | | | |
| | * | | | | | | | | Rename last_paragraph_end variable to a more fitting nameGravatar Numerlor2020-01-01-6/+6
| | | | | | | | | | |
| | * | | | | | | | | Make sure description is truncated to max 1000 charsGravatar Numerlor2019-12-31-2/+8
| |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge pull request #712 from python-discord/audio-whitelistGravatar Leon Sandøy2019-12-30-0/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Update filetype whitelist to include audio formats
| * | | | | | | | | | Update filetype whitelist to include audio formatsGravatar Ava2019-12-30-0/+3
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | Discord has a built-in player for all of these and they are really just stripped down videos, which are allowed.
* | | | | | | | | | Use a static discord shield on the readme.Gravatar scragly2019-12-26-1/+1
| | | | | | | | | |
* | | | | | | | | | Merge pull request #683 from python-discord/discord-testers-whitelistGravatar Leon Sandøy2019-12-26-0/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Whitelist Discord Testers invite link
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into discord-testers-whitelistGravatar Matteo Bertucci2019-12-26-351/+928
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #655 from kwzrd/unittest-mentionsGravatar Mark2019-12-23-49/+186
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / |/| | | | | | | | | | Antispam rules: add unit test for mentions, improve unit tests for links and attachments, fix bug in attachments rule
| * | | | | | | | | | Merge branch 'master' into unittest-mentionsGravatar Mark2019-12-23-30/+44
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #706 from python-discord/SebastiaanZ-add-duckiesGravatar Sebastiaan Zeeff2019-12-22-1/+13
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add new duckies to the constants
| * | | | | | | | | | | Add ducky attributes to bot.constantsGravatar Sebastiaan Zeeff2019-12-22-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have added attributes to the Emojis class in `bot.constants` for the newly added ducky emoji constants.
| * | | | | | | | | | | Add ducky constants to make duckpond workGravatar Sebastiaan Zeeff2019-12-22-1/+7
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | I have added the IDs of the new duckies we have to the constants
* | | | | / / / / / / Allow PSD's and more on antimalware whitelistGravatar Leon Sandøy2019-12-22-0/+5
| |_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also includes the following related formats: - .svg - .ai (Adobe Illustrator) - .aep (Adobe After Effects) - .xcf (GIMP
* | | | | | | | | | Merge pull request #703 from python-discord/user-sync-on-user-updateGravatar Sebastiaan Zeeff2019-12-20-29/+26
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | | Properly sync users to the database when they update their information
| * | | | | | | | | Use on_user_update to properly sync users with dbGravatar Sebastiaan Zeeff2019-12-17-29/+26
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's important to us that we keep the information we have about users in the database in sync with the actual user information the bot can observe in our guild. To do this, we relied on the `on_member_update` event listener to synchronize a user's information when an update of the information was detected. However, unfortunately, this does not work for user account information (i.e., the username, avatar, and discriminator of the user). The solution is to use the `on_user_update` event listener to watch for updates in the user settings and to use the `on_member_update` event listener to watch for updates in guild-related information for that user. (We currently only sync the roles the user has.) See: - https://discordpy.readthedocs.io/en/stable/api.html#discord.on_member_update - https://discordpy.readthedocs.io/en/stable/api.html#discord.on_user_update Note: The docs for `discord.py` make it *seem* like the `on_member_update` event does not fire for updates of theusername, discriminator, and avatar attributes. However, experimentation shows that this event *does* fire; it's just that the member objects provided as `before` and `after` to the listener will already have been updated in cache by the `on_user_update` event that fires *before* it. This means that if the only changes made were to the username, avatar, and discriminator, the `on_member_update` event does fire, but with two *equal* Member objects. This makes it appear as if you may be able to use `on_member_update`, since it fires, but it does not actually contain anything useful.
| * | | | | | | | Merge branch 'master' into unittest-mentionsGravatar Matteo Bertucci2019-12-13-558/+2166
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Use OAuth to be Reddit API compliant (#696)Gravatar scragly2019-12-13-11/+88
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use OAuth to be Reddit API compliant Co-authored-by: null <[email protected]>
| * \ \ \ \ \ \ \ \ Merge branch 'master' into revert-695-revert-510-reddit-api-oauthGravatar scragly2019-12-13-27/+233
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge pull request #694 from python-discord/enh/mod/534/no-ping-dm-failGravatar Mark2019-12-12-3/+6
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Moderation: show emoji for DM failure instead of mentioning actor
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into enh/mod/534/no-ping-dm-failGravatar Mark2019-12-12-258/+604
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #692 from python-discord/enh/verification/688/allow-mods-cmdsGravatar Mark2019-12-12-5/+11
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Allow mods+ to use commands and delete bots' messages in checkpoint channel
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into enh/verification/688/allow-mods-cmdsGravatar Mark2019-12-12-253/+593
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Clean command improvements (#691)Gravatar scragly2019-12-13-16/+46
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Clean command improvements
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into enhancement/690-clean-cmdGravatar scragly2019-12-13-46/+297
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #680 from ↵Gravatar scragly2019-12-13-3/+170
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/Write-unit-tests-for-`bot/utils/time.py` Added `unittest` for `bot.utils.time`
| * | | | | | | | | | | | | Fixed a typo ( due to poor copy pasta and eyeballing skills )Gravatar Shirayuki Nekomata2019-12-13-1/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Added tests for `until_expiration`Gravatar Shirayuki Nekomata2019-12-13-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Similar to `format_infraction_with_duration` ( if not outright copying it ), added 3 tests for `until_expiration`: - None `expiry`. - Custom `max_units`. - Normal use cases.
| * | | | | | | | | | | | | Adding an optional argument for `until_expiration`, update typehints for ↵Gravatar Shirayuki Nekomata2019-12-13-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `format_infraction_with_duration` - `until_expiration` was being a pain to unittests without a `now` ( default to `datetime.utcnow()` ). Adding an optional argument for this will not only make writing tests easier, but also allow more control over the helper function should we need to calculate the remaining time between two dates in the past. - Changed typehint for `date_from` in `format_infraction_with_duration` to `Optional[datetime.datetime]` to better reflect what it is.
| * | | | | | | | | | | | | Merge branch 'master' into Write-unit-tests-for-`bot/utils/time.py`Gravatar scragly2019-12-12-1/+32
| |\ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into Write-unit-tests-for-`bot/utils/time.py`Gravatar scragly2019-12-12-240/+358
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Removed `setUp()` from `TimeTests` since it is not being used for anything.Gravatar Shirayuki Nekomata2019-12-05-3/+0
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Moved all individual test cases into iterables and test with `self.subTest` ↵Gravatar Shirayuki Nekomata2019-12-05-14/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | context manager.
| * | | | | | | | | | | | | | | Added `self.subTest` for tests with multiple test cases & simplified single ↵Gravatar Shirayuki Nekomata2019-12-05-19/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test case tests.
| * | | | | | | | | | | | | | | Introduced test for `test_format_infraction`, refactored ↵Gravatar Shirayuki Nekomata2019-12-05-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `test_parse_rfc1123`, fixed typo.
| * | | | | | | | | | | | | | | Splitting test cases for `format_infraction_with_duration` into proper, ↵Gravatar Shirayuki Nekomata2019-12-05-7/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | independent tests.
| * | | | | | | | | | | | | | | Changed `assert` to `self.assertIs` for `test_wait_until`Gravatar Shirayuki Nekomata2019-12-05-1/+1
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Added missing docstring for `test_humanize_delta_raises_for_invalid_max_units`Gravatar Shirayuki Nekomata2019-12-05-0/+1
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Splitting test cases for `humanize_delta` into proper, independent tests.Gravatar Shirayuki Nekomata2019-12-05-7/+21
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Added `unittest` for `bot.utils.time`Gravatar Shirayuki Nekomata2019-12-04-0/+87
| | | | | | | | | | | | | | | |
| | | | * | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into enhancement/690-clean-cmdGravatar MarkKoz2019-12-12-191/+250
| | | | |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | |_|_|_|_|_|_|_|/ / / | | | | | |/| | | | | | | | | |
| | | | * | | | | | | | | | | | Clean: reformat argumentsGravatar MarkKoz2019-12-11-8/+33
| | | | | | | | | | | | | | | |