aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | | 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
| | | |
* | | | Branding: gate sync via helper functionGravatar kwzrd2021-03-13-8/+19
| | | | | | | | | | | | | | | | Sync make also be invoked with a command; avoid logic duplication.
* | | | Branding: implement daemonGravatar kwzrd2021-03-13-3/+109
| | | |
* | | | Branding: implement internal utilityGravatar kwzrd2021-03-13-2/+174
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the core logic of branding management. In comparison with the previous version, we now maintain all state in Redis, which allows the bot to seamlessly restart without losing any information. The 'send_info_embed' function is intentionally implemented with the consideration of allowing users to invoke it on-demand. It always reads information from the cache, even if the caller could pass a 'MetaFile' instance. So while this may look needlessly indirect right now, it should begin to make sense once the command API is implemented.
* | | | Branding: expose SHA on remote objectsGravatar kwzrd2021-03-13-0/+1
| | | |
* | | | Branding: implement asset application logicGravatar kwzrd2021-03-13-0/+51
| | | |
* | | | Branding: do not require 'RemoteObject' instance to fetch fileGravatar kwzrd2021-03-13-5/+5
| | | |
* | | | Branding: correctly annotate optional attributeGravatar kwzrd2021-03-13-1/+1
| | | |
* | | | Branding: make event instances aware of their locationGravatar kwzrd2021-03-13-2/+6
| | | | | | | | | | | | | | | | This allows us to add a neat string representation.
* | | | Branding: add event gettersGravatar kwzrd2021-03-13-0/+51
| | | | | | | | | | | | | | | | These methods form the API to the repository abstraction.
* | | | Branding: define event construction methodologyGravatar kwzrd2021-03-13-0/+90
| | | |
* | | | Branding: add HTTP fetch helper methodsGravatar kwzrd2021-03-13-0/+54
| | | |
* | | | Branding: migrate constantsGravatar kwzrd2021-03-13-11/+11
| | | | | | | | | | | | | | | | | | | | Constants will only be used in one place and there's not enough of them to warrant a separate module.
* | | | Branding: initiate repository abstractionGravatar kwzrd2021-03-13-0/+15
| | | |
* | | | Branding: remove current implementationGravatar kwzrd2021-03-13-812/+32
| | | | | | | | | | | | | | | | | | | | Since we're planning substantial changes, it will be easier to build from scratch.
* | | | Pipenv: add 'python-frontmatter' & re-lockGravatar kwzrd2021-03-07-48/+59
|/ / /
* | | Remove trailing whitespaceGravatar Boris Muratov2021-03-06-1/+1
| | |
* | | Filtering hotfixGravatar Boris Muratov2021-03-06-1/+7
| | | | | | | | | Bug caused by an outdated function signature in a previous commit in the #1402 PR
* | | Merge pull request #1402 from python-discord/ks123/watchlist-trigger-reasonGravatar Xithrius2021-03-06-16/+44
|\ \ \ | | | | | | | | Implement showing filterlist entry comment in alerts
| * \ \ Merge branch 'master' into ks123/watchlist-trigger-reasonGravatar Xithrius2021-03-06-501/+932
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1392 from ↵Gravatar Xithrius2021-03-06-3/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | ChrisLovering/Don't-suggest-when-a-tag-is-on-cooldown Don't fuzzy search for tags when tag is on cooldown
| * \ \ \ Merge branch 'master' into Don't-suggest-when-a-tag-is-on-cooldownGravatar Xithrius2021-03-06-515/+1010
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1449 from python-discord/json-loggingGravatar Joe Banks2021-03-06-114/+196
|\ \ \ \ \
| * | | | | Version lock JSON loggerGravatar Joe Banks2021-03-06-2/+2
| | | | | |
| * | | | | Use JSON logging when debug mode is disabledGravatar Joe Banks2021-03-06-16/+33
| | | | | |
| * | | | | Add JSON logging dependenciesGravatar Joe Banks2021-03-06-98/+163
|/ / / / /