aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | | | | Incidents: only `add_signals` if missingGravatar kwzrd2020-06-12-2/+10
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Incidents: make `on_message` ignore pinned messagesGravatar kwzrd2020-06-12-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is now necessary as we call the listener ourselves from the crawl task. An already existing, pinned message, can be received.
| * | | | | | | | | | | | | | | Incidents: implement & schedule `crawl_incidents` taskGravatar kwzrd2020-06-12-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See docstring for further information. This will run on start-up to retroactively add missing emoji. Ratelimit-wise this should be fine, as there should never be too many missing emoji.
| * | | | | | | | | | | | | | | Incidents: implement `on_message` listener & guardsGravatar kwzrd2020-06-12-3/+20
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Incidents: implement `add_signals` helperGravatar kwzrd2020-06-12-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Looks like it can be static, at least for now.
| * | | | | | | | | | | | | | | Incidents: define method stubs for message event handlingGravatar kwzrd2020-06-12-0/+10
| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Incidents: define allowed roles and emojiGravatar kwzrd2020-06-11-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These serve as whitelists, i.e. any reaction using an emoji not explicitly allowed, or from a user not specifically allowed, will be rejected. Such reactions will be removed by the bot.
| * | | | | | | | | | | | | | | Merge branch 'origin/master' into kwzrd/incidentsGravatar kwzrd2020-06-11-53/+115
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Incidents: add #incidents-archive webhook constantGravatar kwzrd2020-06-11-1/+2
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Incidents: add #incidents-archive channel constantGravatar kwzrd2020-06-11-0/+2
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Incidents: create Signal enum & link members with emojisGravatar kwzrd2020-06-08-0/+10
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Incidents: add new emoji constantsGravatar kwzrd2020-06-08-0/+8
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Incidents: add incidents module & new ext boilerplateGravatar kwzrd2020-06-08-1/+17
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Merge branch 'role-reminders' of github.com:python-discord/bot into ↵Gravatar kosayoda2020-07-19-0/+1
| | | | | | | | | | | | | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | role-reminders
| | | | | | | | | | | | | | * | | | Add whitespace to improve readabilityGravatar Kieran Siek2020-07-19-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]>
| | | | | | | | | | | | | * | | | | Revert "Remove duplicate reminder deletion."Gravatar kosayoda2020-07-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 776b4379c478284803a4a526b5f14fe63d8e7c01. This is already being fixed in #835, and therefore is no longer required.
| | | | | | | | | | | | | * | | | | Remove duplicate mentions from reminder argumentsGravatar kosayoda2020-07-19-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also accounts for the author passing themselves to mention, and therefore avoids the user from being told they're not allowed to mention themselves even though they could.
| | | | | | | | | | | | | * | | | | Refactor commands return typeGravatar kosayoda2020-07-19-5/+8
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | | Move mentions validation to another methodGravatar kosayoda2020-07-19-13/+20
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | | Refactor reminder edits to reduce code duplicationGravatar kosayoda2020-07-19-40/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reminder expiration returnedfrom the API call is also now parsed again even when the edit is to the duration since it does not matter and trying to keep it DRY while still doing that check is a pain.
| | | | | | | | | | | | | * | | | | Rename method to improve readabilityGravatar kosayoda2020-07-19-3/+3
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | | Fix incorrect typehint and shorten method nameGravatar kosayoda2020-07-19-3/+3
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | | Extract `send_denial` to a utility functionGravatar kosayoda2020-07-19-17/+20
| | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | | Namespace Member and Role to avoid extra importsGravatar kosayoda2020-07-19-3/+2
| | | | | | | | | | | | | |/ / / /
| | | | | | | | | | | | | * | | | Allow editing additional mentions for reminders.Gravatar kosayoda2020-07-16-0/+28
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | List additional mentions in `!reminder list`.Gravatar kosayoda2020-07-16-3/+9
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Send additional mentions in reminders.Gravatar kosayoda2020-07-16-1/+5
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Allow mentioning other users and roles in reminders.Gravatar kosayoda2020-07-16-6/+26
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Add generator that converts IDs to Role or Member objects.Gravatar kosayoda2020-07-16-0/+7
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Add method to check if user is allowed to mention in reminders.Gravatar kosayoda2020-07-16-1/+21
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Extract sending error embed to a separate method.Gravatar kosayoda2020-07-16-13/+12
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | | Remove duplicate reminder deletion.Gravatar kosayoda2020-07-16-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function `_delete_reminder` was called twice, once in `schedule_reminder`, which calls `send_reminder`, then another in `send_reminder` itself. This led to a 404 response from the site api, as the reminder was already deleted the first time. Fixes BOT-6W
| | | | | | | | | | | | | | * | | Jam Tests: space out lines for readabilityGravatar MarkKoz2020-07-22-0/+6
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: remove default_args attributeGravatar MarkKoz2020-07-22-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Kind of redundant since it's only used by two tests.
| | | | | | | | | | | | | | * | | Jam Tests: re-arrange tests to follow definition order in the cogGravatar MarkKoz2020-07-22-10/+10
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: fix utils patchGravatar MarkKoz2020-07-22-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | stop needs to be called on the patcher, not the mock. Furthermore, using addCleanup is safer than tearDown because the latter may not be called if an exception is raised in setUp.
| | | | | | | | | | | | | | * | | Jam Tests: Fix `test_duplicate_member_provided` assertionsGravatar ks1292020-06-13-1/+3
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Fix wrong function name and convert them to mocksGravatar ks1292020-06-13-1/+3
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Simplify and make tests more secureGravatar ks1292020-06-13-6/+4
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jams: Fix `get_overwrites` return typeGravatar ks1292020-06-13-2/+2
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Update `Context` to `Guild` for tests tooGravatar ks1292020-06-13-19/+19
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jams: Use `Guild` instead `Context` for helper functionsGravatar ks1292020-06-13-19/+19
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Fix `create_channels`, `get_category` docstringsGravatar ks1292020-06-13-2/+2
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Apply recent command splitting to `test_jam_roles_adding`Gravatar ks1292020-06-11-2/+2
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Remove unnecessary `Context` mock resetsGravatar ks1292020-06-11-2/+0
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Implement default argumentsGravatar ks1292020-06-11-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To avoid repeating same arguments, added default arguments that is unpacked on function call.
| | | | | | | | | | | | | | * | | Jam Tests: Apply recent changes to `test_category_dont_exist`Gravatar ks1292020-06-11-2/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Simplify and update `test_category_channel_exist`Gravatar ks1292020-06-11-3/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Merge text and voice channel creation testsGravatar ks1292020-06-11-21/+24
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Jam Tests: Apply recent changes to overwrites testGravatar ks1292020-06-11-2/+1
| | | | | | | | | | | | | | | | |