aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | 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
| | | | | | | | | | | | | | | |
| | | | * | | | | | | | | | | | Clean: allow amount argument to be skipped Gravatar Mark2019-12-11-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This make the channel specifiable without the amount. Co-Authored-By: scragly <[email protected]>
| | | | * | | | | | | | | | | | Clean: support specifying a channel different than the context'sGravatar MarkKoz2019-12-11-13/+18
| | | | | | | | | | | | | | | |
| | | | * | | | | | | | | | | | Clean: un-hide from help and add purge aliasGravatar MarkKoz2019-12-11-1/+1
| | | | | |_|_|_|_|_|/ / / / / | | | | |/| | | | | | | | | |
| | | | | | * | | | | | | | | Verification: check channel before checking for bot messagesGravatar MarkKoz2019-12-12-3/+3
| | | | | | | | | | | | | | |
| | | | | | * | | | | | | | | Verification: delete bots' messages (#689)Gravatar MarkKoz2019-12-11-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Messages are deleted after a delay of 10 seconds. This helps keep the channel clean. The periodic ping is an exception; it will remain.
| | | | | | * | | | | | | | | Verification: allow mods+ to use commands in checkpoint (#688)Gravatar MarkKoz2019-12-11-2/+3
| | | | | |/ / / / / / / / / | | | | |/| | | | | | | | |
| | | | | | * | | | | | | | Moderation: show emoji for DM failure instead of mentioning actor (#534)Gravatar MarkKoz2019-12-11-3/+6
| | | | | |/ / / / / / / / | | | | |/| | | | | | | |
| | | | | | * | | | | | | Revert "Revert "Use OAuth to be Reddit API compliant""Gravatar Joseph2019-12-13-11/+88
| |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Revert "Use OAuth to be Reddit API compliant" (#695)Gravatar Joseph2019-12-13-88/+11
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Use OAuth to be Reddit API compliant"
| * | | | | | | | | | | | Revert "Use OAuth to be Reddit API compliant"Gravatar Joseph2019-12-13-88/+11
|/ / / / / / / / / / / /
* | | | | | | | | | | | Use OAuth to be Reddit API compliant (#510)Gravatar scragly2019-12-13-11/+88
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use OAuth to be Reddit API compliant Co-authored-by: Jens <[email protected]> Co-authored-by: Mark <[email protected]> Co-authored-by: null <[email protected]>
| * | | | | | | | | | | Merge branch 'master' into reddit-api-oauthGravatar scragly2019-12-13-237/+383
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Display time left until expiration of infraction (#679)Gravatar scragly2019-12-12-1/+32
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Display time left until expiration of infraction Co-authored-by: null <[email protected]>
| * | | | | | | | | | | Merge branch 'master' into Display-time-left-until-expiration-of-infractionGravatar scragly2019-12-12-240/+358
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #693 from python-discord/enh/mod/624/edit-recent-infractionGravatar Mark2019-12-12-19/+49
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Allow "recent" as infraction ID for infraction edit command
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into enh/mod/624/edit-recent-infractionGravatar Mark2019-12-12-26/+49
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Token and bad code (#500)Gravatar scragly2019-12-12-26/+49
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Token and bad code Co-authored-by: Daniel Brown <[email protected]> Co-authored-by: Mark <[email protected]>
| * | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into token_and_bad_codeGravatar MarkKoz2019-12-12-1169/+2555
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Requested changeGravatar kraktus2019-12-05-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include the check about whether or not there is a token in the posted message in `parse_codeblock` boolean.