aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Update information tests for new embed color logicGravatar Ben Soyka2021-04-04-6/+8
|
* Use Member.colour rather than a list comprehension for !userGravatar Ben Soyka2021-04-04-3/+1
|
* Ignore colourless roles in !user embedGravatar Ben Soyka2021-04-04-1/+3
|
* Merge pull request #1497 from python-discord/mbaruh-staff_infoGravatar Boris Muratov2021-04-04-0/+2
|\ | | | | Blacklist staff_info for duckpond
| * Blacklist staff_info for duckpondGravatar Boris Muratov2021-04-04-0/+2
|/
* Merge pull request #1464 from python-discord/allow-poll-to-leadsGravatar ks1292021-04-01-2/+6
|\ | | | | Allow !poll to leads
| * Merge branch 'main' into allow-poll-to-leadsGravatar ks1292021-04-01-1151/+1276
| |\ | |/ |/|
* | Merge pull request #1492 from python-discord/fix-delete-closing-reason-statGravatar Matteo Bertucci2021-04-01-1/+2
|\ \ | | | | | | Use deleted reason if help channel is closed due to being empty
| * \ Merge branch 'main' into fix-delete-closing-reason-statGravatar Matteo Bertucci2021-04-01-791/+774
| |\ \ | |/ / |/| |
* | | Merge pull request #1494 from python-discord/policy-documentsGravatar Matteo Bertucci2021-04-01-122/+8
|\ \ \ | | | | | | | | Update our policy documents
| * | | Update policy documentsGravatar Matteo Bertucci2021-04-01-122/+8
|/ / /
* | | Merge pull request #1463 from kwzrd/kwzrd/brandingGravatar Joe Banks2021-03-31-669/+766
|\ \ \
| * | | Branding: log after successful fetchGravatar kwzrd2021-03-31-0/+4
| | | | | | | | | | | | | | | | | | | | Co-authored-by: Shivansh-007 <[email protected]> Co-authored-by: Joe Banks <[email protected]>
| * | | Branding: apply documentation improvements after reviewGravatar kwzrd2021-03-31-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | No code changes in this commit. Co-authored-by: Shivansh-007 <[email protected]> Co-authored-by: Joe Banks <[email protected]>
| * | | Branding: target 'main' branchGravatar kwzrd2021-03-31-2/+2
| | | | | | | | | | | | | | | | | | | | With the branding-side PR merged, we can now target the production branch.
| * | | Merge: changes from 'upstream/main'Gravatar kwzrd2021-03-31-272/+282
| |\ \ \ | |/ / / |/| | | | | | | Lockfile conflict resolved by re-locking on the merged Pipfile.
| * | | Merge: changes from 'upstream/main' & conflict resolutionGravatar kwzrd2021-03-28-399/+854
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflict in the lockfile resolved by re-locking the merged Pipfile. Conflict in Branding constants resolved by keeping my local version. Change in the cog's target branch to 'main' from 'master' is currently irrelevant as we targets a development branch anyway.
| * | | | Branding: add contextual message to #changelog notificationsGravatar kwzrd2021-03-28-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | It would be strange to just send the embed with no explanation of what it means.
| * | | | Branding: omit notification when entering evergreenGravatar kwzrd2021-03-27-3/+3
| | | | | | | | | | | | | | | | | | | | The fallback event should not produce a notification.
| * | | | Branding: revise log messages & levelsGravatar kwzrd2021-03-27-49/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Logs are now proper sentences ended with full stops. Exceptions are logged with full tracebacks, and log level are revised to be more sensible and consistent across the extension.
| * | | | Branding: revise documentationGravatar kwzrd2021-03-27-97/+80
| | | | |
| * | | | Branding: ensure daemon logs exceptionsGravatar kwzrd2021-03-27-5/+8
| | | | |
| * | | | Branding: make 'meta.md' parser synchronousGravatar kwzrd2021-03-26-2/+2
| | | | | | | | | | | | | | | | | | | | No reason for this to be async.
| * | | | Branding: raise custom error when constructing remote objectsGravatar kwzrd2021-03-26-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | The default KeyError message from dict lookup is just the missing key. In order to give more context in the log message, we raise our own.
| * | | | Branding: raise on non-200 responsesGravatar kwzrd2021-03-26-23/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fetch helpers will now raise when the request fails rather than logging a warning and returning a fallback value. This allows better error logging as the caller is able to log the propagated exception while adding its own context. Additionally, the caller in some cases no longer needs to check for the None return and raise its own exception.
| * | | | Branding: cache fresh event description in daemonGravatar kwzrd2021-03-25-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the event description & duration strings were only stored on event entry. In the case that the description or duration change for an on-going event, the cached values wouldn't be updated. After this commit, the cache is refreshed daily by the daemon.
| * | | | Branding: improve 'compound_hash' docstringGravatar kwzrd2021-03-25-1/+5
| | | | |
| * | | | Branding: update 'synchronise' docsGravatar kwzrd2021-03-18-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After previous changes, the docstring was no longer accurate. See: 1d5625a2f47a1d4d050f9eb0eb7a18e7d6fe171b
| * | | | Branding: extend command aliasesGravatar kwzrd2021-03-17-2/+2
| | | | |
| * | | | Branding: adjust daemon start-up behaviourGravatar kwzrd2021-03-17-35/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The daemon will now perform a sync iteration immediately when started, and then every UTC midnight. Previously, it would only perform the initial iteration when started for the first time, which is odd. It is also believed that splitting the daemon's logic into three separate functions is beneficial: before, loop, and main. This commit makes log and doc adjustments where appropriate.
| * | | | Pipenv: bump 'python-frontmatter' pin & re-lockGravatar kwzrd2021-03-15-19/+35
| | | | | | | | | | | | | | | | | | | | Fresh stable release, just in time!
| * | | | Branding: decode 'meta.md' using UTF-8Gravatar kwzrd2021-03-15-1/+1
| | | | |
| * | | | Branding: suppress 'ValueError' on empty-dict updateGravatar kwzrd2021-03-14-4/+6
| | | | |
| * | | | Branding: check for empty icon cacheGravatar kwzrd2021-03-14-0/+4
| | | | |
| * | | | Branding: add embed length cut-offs for safetyGravatar kwzrd2021-03-14-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should never do anything, but it's better to be safe. Values taken from Discord developer docs.
| * | | | Branding: provide class documentationGravatar kwzrd2021-03-14-2/+43
| | | | |
| * | | | Branding: log correct amount of cached eventsGravatar kwzrd2021-03-14-1/+3
| | | | |
| * | | | Branding: log event path alongside errorGravatar kwzrd2021-03-14-1/+1
| | | | | | | | | | | | | | | | | | | | Knowing which event failed would probably be quite useful.
| * | | | Branding: avoid sending #changelog notification on resyncGravatar kwzrd2021-03-14-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The notification is now sent conditionally depending on whether we're entering a new event. This prevents sending a repeating notification in the case of a manual resynchronisation. A practical example of when this may trigger is when a staff member temporarily applies custom assets & then uses the sync command to reapply the current event.
| * | | | Branding: make 'cog_unload' synchronousGravatar kwzrd2021-03-14-1/+1
| | | | | | | | | | | | | | | | | | | | Discord.py doesn't await the return value.
| * | | | Branding: show success information in 'sync' responseGravatar kwzrd2021-03-14-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | | Now that the boolean flags are propagating from 'apply_asset', we can present them to the user.
| * | | | Branding: propagate success-indicating boolean from 'apply_asset'Gravatar kwzrd2021-03-14-19/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The sync command will now be able to use present this information to the invoking user. This commit also prevents the cached banner & icon hash from being overwritten in the case of asset upload failure. As a result, the daemon will attempt to re-apply the assets the following day.
| * | | | Branding: do not call 'rotate_icons' from rotation initGravatar kwzrd2021-03-14-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It makes more sense for the init and the rotation to be separate operations. In a subsequent commit, the separation of responsibility will allow the `rotate_icons` function to have a meaningful return value.
| * | | | Branding: replace ugly lambda with 'attrgetter'Gravatar kwzrd2021-03-13-1/+2
| | | | |
| * | | | Branding: add calendar command groupGravatar kwzrd2021-03-13-0/+63
| | | | |
| * | | | Branding: cache all available eventsGravatar kwzrd2021-03-13-11/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a prequel to adding a calendar command. To avoid re-querying the branding repo on command invocation, event information will be cached whenever we make requests. The command can then simply get an up-to-date event schedule from the cache, with the option of forcing an update via the 'populate_cache_events' function. Since we cannot easily serialize entire 'Event' instances, we simply store what's needed - the event name, and its duration. The author has verified that the cache maintains order; in this case chronological order based on event start date.
| * | | | Branding: extract duration string in helper functionGravatar kwzrd2021-03-13-9/+17
| | | | |
| * | | | Branding: add missing early exitGravatar kwzrd2021-03-13-0/+1
| | | | |
| * | | | Branding: lock commands to mods+ where necessaryGravatar kwzrd2021-03-13-1/+3
| | | | |
| * | | | Branding: implement command interfaceGravatar kwzrd2021-03-13-1/+85
| | | | |