| Commit message (Collapse) | Author | Age | Lines | ||
|---|---|---|---|---|---|
| ... | |||||
| | | | | | | | | | | | | | | * | | | | Add whitespace to improve readability | 2020-07-19 | -0/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]> | ||||
| | | | | | | | | | | | | | * | | | | | Revert "Remove duplicate reminder deletion." | 2020-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 arguments | 2020-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 type | 2020-07-19 | -5/+8 | ||
| | | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | | Move mentions validation to another method | 2020-07-19 | -13/+20 | ||
| | | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | | Refactor reminder edits to reduce code duplication | 2020-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 readability | 2020-07-19 | -3/+3 | ||
| | | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | | Fix incorrect typehint and shorten method name | 2020-07-19 | -3/+3 | ||
| | | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | | Extract `send_denial` to a utility function | 2020-07-19 | -17/+20 | ||
| | | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | | Namespace Member and Role to avoid extra imports | 2020-07-19 | -3/+2 | ||
| | | | | | | | | | | | | | |/ / / / | |||||
| | | | | | | | | | | | | | * | | | | Allow editing additional mentions for reminders. | 2020-07-16 | -0/+28 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | List additional mentions in `!reminder list`. | 2020-07-16 | -3/+9 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Send additional mentions in reminders. | 2020-07-16 | -1/+5 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Allow mentioning other users and roles in reminders. | 2020-07-16 | -6/+26 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Add generator that converts IDs to Role or Member objects. | 2020-07-16 | -0/+7 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Add method to check if user is allowed to mention in reminders. | 2020-07-16 | -1/+21 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Extract sending error embed to a separate method. | 2020-07-16 | -13/+12 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | * | | | | Remove duplicate reminder deletion. | 2020-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 readability | 2020-07-22 | -0/+6 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: remove default_args attribute | 2020-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 cog | 2020-07-22 | -10/+10 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: fix utils patch | 2020-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` assertions | 2020-06-13 | -1/+3 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Fix wrong function name and convert them to mocks | 2020-06-13 | -1/+3 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Simplify and make tests more secure | 2020-06-13 | -6/+4 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Fix `get_overwrites` return type | 2020-06-13 | -2/+2 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Update `Context` to `Guild` for tests too | 2020-06-13 | -19/+19 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Use `Guild` instead `Context` for helper functions | 2020-06-13 | -19/+19 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Fix `create_channels`, `get_category` docstrings | 2020-06-13 | -2/+2 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Apply recent command splitting to `test_jam_roles_adding` | 2020-06-11 | -2/+2 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Remove unnecessary `Context` mock resets | 2020-06-11 | -2/+0 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Implement default arguments | 2020-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` | 2020-06-11 | -2/+1 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Simplify and update `test_category_channel_exist` | 2020-06-11 | -3/+1 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Merge text and voice channel creation tests | 2020-06-11 | -21/+24 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Apply recent changes to overwrites test | 2020-06-11 | -2/+1 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Add more assertions to result message sending test | 2020-06-11 | -1/+7 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Make early exiting test more secure | 2020-06-11 | -2/+6 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Convert some functions to staticmethod | 2020-06-11 | -3/+6 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Move roles adding to another function from inside of command | 2020-06-11 | -8/+11 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Change return plain text to channel mention in `create_channels` | 2020-06-11 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Move channels creation to new function instead inside command | 2020-06-11 | -19/+26 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Move overwrites generation to outside of command | 2020-06-11 | -22/+28 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jams: Move category checking and creation to another function | 2020-06-11 | -16/+22 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Use class member of patch instead decorator on most of tests | 2020-06-11 | -21/+19 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Fix `test_result_sending` docstring | 2020-06-11 | -1/+1 | ||
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Mark <[email protected]> | ||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Create test for `setup` function | 2020-06-11 | -1/+11 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Create test for successful `ctx.send` calling | 2020-06-11 | -0/+6 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Create test for team jam roles adding | 2020-06-11 | -0/+14 | ||
| | | | | | | | | | | | | | | | | | | |||||
| | | | | | | | | | | | | | | * | | | Jam Tests: Create test for team voice channel creating | 2020-06-11 | -0/+12 | ||
| | | | | | | | | | | | | | | | | | | |||||