aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge pull request #782 from python-discord/feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-163/+229
|\ | | | | Rename constants to match their guild counterparts
| * Merge branch 'master' into feat/backend/b496/rename-constantsGravatar Leon Sandøy2020-02-28-33/+65
| |\ | |/ |/|
* | Merge pull request #798 from python-discord/bug/mod/bot-1v/infr-edit-task-cancelGravatar Sebastiaan Zeeff2020-02-27-1/+3
|\ \ | | | | | | Moderation: fix task cancellation for permanent infraction when editing
| * | Moderation: fix task cancellation for permanent infraction when editingGravatar MarkKoz2020-02-27-1/+3
|/ / | | | | | | | | | | | | A task should not be cancelled if an infraction is permanent because tasks don't exist for permanent infractions. Fixes BOT-1V
* | Merge pull request #785 from Numerlor/channel_constants_checkGravatar Mark2020-02-27-29/+53
|\ \ | | | | | | Verify channels against server and reddit OAuth2 in constants on startup.
| * \ Merge branch 'master' into channel_constants_checkGravatar Mark2020-02-27-9/+20
| |\ \ | |/ / |/| |
* | | Merge pull request #790 from ks129/masterGravatar S. Co12020-02-26-2/+8
|\ \ \ | | | | | | | | Added ignoring staff to Antimalware check.
| * | | Add clarifying comment to role checking logic implementationGravatar S. Co12020-02-26-0/+1
| | | |
| * | | Merge branch 'master' into masterGravatar S. Co12020-02-26-7/+12
| |\ \ \ | |/ / / |/| | |
* | | | Disable TRACE logging for Sentry breadcrumbs.Gravatar Joseph2020-02-26-1/+1
| | | |
| * | | Added DMs ignoring to antimalware checkGravatar ks1232020-02-26-1/+2
| | | |
| * | | Added to AntiMalware staff ignore check.Gravatar Karlis. S2020-02-26-1/+5
| | | |
| | * | Fix comparison operator when checking token expiration.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | With `<` the check only went through when the token was already expired, making revoking redundant; and didn't go through when the token still had some time before expiration.
| | * | Check for falsy values instead of ``""` and `None` explicitly.Gravatar Numerlor2020-02-26-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | After the change to also check empty strings to avoid unucessary requests, it is no longer necessary to do an explicit value check, as the only values that can come from the .env file are `None` and strings Co-authored-by: Karlis S <[email protected]>
| | * | Specify the logged time is in UTC.Gravatar Numerlor2020-02-26-1/+1
| | | |
| | * | Create task for `revoke_access_token` when unloading cog to ensure it's ↵Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | executed.
| | * | Make sure token exists before checking its expiration.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | Without the check and an invalid token, an AttributeError is raised; blocking the relevant ClientError from being raised in `get_access_token`.
| | * | Check for empty strings alongside None before loading cog.Gravatar Numerlor2020-02-26-1/+2
| | | | | | | | | | | | | | | | Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.
| | * | Restyle if body to include the error instead of adding the cog.Gravatar Numerlor2020-02-25-3/+3
| | | |
| | * | Fix `__iter__` for classes without subsections.Gravatar Numerlor2020-02-25-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | The previous implementation assumed the config class was a subsection, failing with a KeyError if it wasn't one. Co-authored-by: kwzrd <[email protected]>
| | * | Assign created task to a variable.Gravatar Numerlor2020-02-25-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: SebastiaanZ <[email protected]>
| | * | Wait for available guild instead of bot startup.Gravatar Numerlor2020-02-25-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: SebastiaanZ <[email protected]>
| | * | Merge branch 'master' into channel_constants_checkGravatar Numerlor2020-02-25-551/+1704
| | |\ \ | | |/ / | |/| |
| | * | Do not attempt to load Reddit cog when environment variables are not provided.Gravatar Numerlor2020-02-24-1/+4
| | | | | | | | | | | | | | | | When environment variables weren't provided; the cog attempted to create a BasicAuth object with None as values resulting in an exception before the event loop was started and a subsequent crash.
| | * | Remove unused `chunks` function and its tests.Gravatar Numerlor2020-02-24-26/+1
| | | | | | | | | | | | | | | | The function was only used in the since removed `Events` cog.
| | * | Change docstring mood.Gravatar Numerlor2020-02-24-1/+1
| | | |
| | * | Change warning text.Gravatar Numerlor2020-02-24-1/+1
| | | |
| | * | Add ConfigVerifier cog.Gravatar Numerlor2020-02-24-0/+41
| | | | | | | | | | | | | | | | Adds ConfigVerifier which verifies channels when loaded.
| | * | Implement `__iter__` on constants YAMLGetter.Gravatar Numerlor2020-02-24-0/+4
| | | | | | | | | | | | | | | | Python tries to fall back on passing indices to `__getitem__` without iter implemented; failing on the first line.
| | | * Fix mismatched constant names in syncer testsGravatar S. Co12020-02-25-6/+6
| | | |
| | | * Configure staff role & channel groupings in YAMLGravatar S. Co12020-02-25-10/+27
| | | | | | | | | | | | Delete duplicate keys that were missed in the merge
| | | * Merge branch 'master' into feat/backend/b496/rename-constantsGravatar S. Co12020-02-25-546/+1676
| | | |\ | |_|_|/ |/| | |
* | | | Merge pull request #781 from ↵Gravatar S. Co12020-02-25-6/+11
|\ \ \ \ | |/ / / |/| | | | | | | | | | | python-discord/bug/utils/bot-1c/reminder-unschedule-failure Reminders: don't cancel task if reminder is invalid when rescheduling
| * | | Merge branch 'master' into bug/utils/bot-1c/reminder-unschedule-failureGravatar Joseph2020-02-25-551/+1704
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #711 from python-discord/bug/backend/b704/ready-missing-cacheGravatar Mark2020-02-23-536/+1657
|\ \ \ \ | | | | | | | | | | Prevent the role syncer from wiping the database table during API latency
| * \ \ \ Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-23-621/+997
| |\ \ \ \ | |/ / / / |/| | | | | | | | | bug/backend/b704/ready-missing-cache
| * | | | Sync: code style refactoringGravatar MarkKoz2020-02-23-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Convert diff namedtuple to dict outside the dict comprehension * Define long condition as a boolean instead of in the if statement * Pass role and user dicts to aiohttp normally instead of unpacking
| * | | | Bot: send empty cache warning to a webhookGravatar MarkKoz2020-02-21-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is more visible than it would be if it was only logged. * Add a webhook for the dev-log channel to constants
| * | | | API: add comment explaining class attributesGravatar MarkKoz2020-02-19-0/+2
| | | | | | | | | | | | | | | | | | | | Explain changes caused by 22a55534ef13990815a6f69d361e2a12693075d5.
| * | | | Sync: add confirmation timeout and max diff to configGravatar MarkKoz2020-02-19-16/+24
| | | | |
| * | | | Tests: fix unawaited error for MockAPIClientGravatar MarkKoz2020-02-12-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This error is due to the use of an actual instance of APIClient as the spec for the mock. recreate() is called in __init__ which in turn creates a task for the _create_session coroutine. The approach to the solution is to use the type for the spec rather than and instance, thus avoiding any call of __init__. However, without an instance, instance attributes will not be included in the spec. Therefore, they are defined as class attributes on the actual APIClient class definition and given default values. Alternatively, a subclass of APIClient could have been made in the tests.helpers module to define those class attributes. However, it seems easier to maintain if the attributes are in the original class definition.
| * | | | Sync tests: use async_test decoratorGravatar MarkKoz2020-02-12-94/+135
| | | | |
| * | | | Sync tests: test on_member_joinGravatar MarkKoz2020-02-12-0/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Should PUT user's data or POST it if the user doesn't exist. ResponseCodeError should be re-raised if status code isn't a 404. A helper method was added to reduce code redundancy between the 2 tests.
| * | | | Sync tests: add a third role with a lower ID to on_member_update testGravatar MarkKoz2020-02-12-1/+2
| | | | | | | | | | | | | | | | | | | | This better ensures that roles are being sorted when patching.
| * | | | Sync tests: fix on_member_remove listener testGravatar MarkKoz2020-02-12-13/+6
| | | | | | | | | | | | | | | | | | | | | | | | | The listener was changed earlier to simply set in_guild to False. This commit accounts for that in the test.
| * | | | Sync tests: assert that listeners are actually added as listenersGravatar MarkKoz2020-02-12-0/+14
| | | | |
| * | | | Sync tests: test sync commands require the admin permissionGravatar MarkKoz2020-02-12-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The sync commands should only run if the author has the administrator permission. * Add missing spaces after class docstrings * Add missing docstring to SyncCogCommandTests
| * | | | Tests: create TestCase subclass with a permissions check assertionGravatar MarkKoz2020-02-12-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The subclass will contain assertions that are useful for testing Discord commands. The currently included assertion tests that a command will raise a MissingPermissions exception if the author lacks permissions.
| * | | | Sync: only update in_guild field when a member leavesGravatar MarkKoz2020-02-12-16/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The member and user update listeners should already be detecting and updating other fields so by the time a user leaves, the rest of the fields should be up-to-date. * Dedent condition which was indented too far
| * | | | Sync tests: test sync users commandGravatar MarkKoz2020-02-12-0/+7
| | | | | | | | | | | | | | | | | | | | sync() should be called on the UserSyncer.
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-04 09:23:25 +0000