aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | Corrected a few typos in the json file and also the following refinements ↵Gravatar RohanJnr2019-10-07-27/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | for the quiz game: New scoreboard for every game. Store overall score board which refreshs when bot restarts.
| | | * Reword the pipenv environment bullet for clarityGravatar S. Co12020-01-10-4/+4
| | | |
| | | * Simplify language for request to implementGravatar scragly2019-12-17-3/+3
| | | | | | | | | | | | Co-Authored-By: Leon Sandøy <[email protected]>
| | | * Use international spelling for BehaviourGravatar scragly2019-12-13-2/+2
| | | |
| | | * Merge enhancement and new feature into a single "feature" issue.Gravatar scragly2019-12-13-65/+39
| | | |
| | | * Swap the comments around to avoid hasty deletion.Gravatar scragly2019-12-12-2/+2
| | | |
| | | * "Possible Alternatives" section deferred to "Additional Details".Gravatar scragly2019-12-12-8/+2
| | | |
| | | * Remove "Possible Issues" sections.Gravatar scragly2019-12-12-8/+0
| | | |
| | | * Add the requirement to allow PR edits from contributors.Gravatar scragly2019-12-12-0/+1
| | | |
| | | * Make clearer that PR description is for implementation details.Gravatar scragly2019-12-12-2/+2
| | | |
| | | * Put additional details and reserve section on all templates.Gravatar scragly2019-12-12-0/+25
| | | |
| | | * Refine the pull request sections and use proper comments.Gravatar scragly2019-12-12-10/+22
| | | |
| | | * Remove leftover automatically generated templates.Gravatar scragly2019-12-12-60/+0
| | | |
| | | * Add issue template config.Gravatar scragly2019-12-12-0/+7
| | | |
| | | * Refine Github TemplatesGravatar scragly2019-12-12-19/+129
| |_|/ |/| |
* | | Merge pull request #327 from AG4lyf/bookmarkGravatar Leon Sandøy2019-12-10-0/+61
|\ \ \ | | | | | | | | [Feature] `.bm` `.pin` bookmarks/personal-pins
| * \ \ Merge branch 'bookmark' of https://github.com/AG4lyf/seasonalbot into bookmarkGravatar kwzrd2019-12-10-8/+58
| |\ \ \
| | * \ \ Merge branch 'master' into bookmarkGravatar Leon Sandøy2019-12-10-8/+58
| | |\ \ \ | |_|/ / / |/| | | |
* | | | | Merge pull request #328 from python-discord/aoc-announcement-mention-fixGravatar Sebastiaan Zeeff2019-12-04-8/+58
|\ \ \ \ \ | | | | | | | | | | | | Make the daily Advent of Code subscription service actually ping subscribers
| * | | | | Unlock AoC role to make announcements actually ping the usersGravatar Sebastiaan Zeeff2019-12-04-7/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Advent of Code cog has a subscription system that allows members to subscribe to a daily notification indicating the puzzle for that day has become available. However, we introduces a more stringent role mentionability policy that meant that the mentions did not actually ping the members subscribed. To solve this, I've made sure that the bot unlocks the role before making the announcement using the `unlock_role` context manager. This also means the role is automatically unlocked after the message is sent. In addition, I noticed that Seasonalbot was consistently announcing the puzzle about 0.5 seconds early. I've correct this by adding a second to the sleep delay. In addition, the bot now verifies that the puzzle is available using a small HEAD request. While this does send a request to the AoC server, it prevents multiple users from sending unnecessary requests by following our link before the puzzle is actually available.
| * | | | | Make AoC channel ID configurable for testing environmentsGravatar Sebastiaan Zeeff2019-12-04-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have made the `#advent-of-code` channel id configurable using the same environment variable technique used for other settings. This makes it easier to test features that rely on this channel in a test environment.
| * | | | | Add context manager to safely unlock role mentionabilityGravatar Sebastiaan Zeeff2019-12-04-0/+23
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, our regualar roles are not mentionable by default. This means that features that rely on roles to keep track of users that want to receive announcements, like the AoC Day Countdown, don't actually ping the users subscribed to it. The solution is obviously that the bot should unlock prior to making the announcement. However, this is complicated by the fact that there needs to be a sufficient delay. both between unlocking and sending the message and between sending the message and locking the role again. If not, Discord's not done synchronizing across all servers and some users won't receive a ping. To make this easier, I have implemented a context manager that takes an instance of `discord.Role` and an optional `delay` (default: 5s) that yields a context in which the role is unlocked. This context manager also makes sure that the role is locked even if an exception occured within the unlocked context.
| * | | | Use bookmark icon url from constantsGravatar kwzrd2019-12-10-2/+2
| | | | |
| * | | | Add bookmark icon url to constantsGravatar kwzrd2019-12-10-0/+6
| |/ / /
| * | | Refactor function signatureGravatar kwzrd2019-12-10-4/+5
| | | | | | | | | | | | | | | | Code style is now consistent with the bot repo.
| * | | Adjust hyperlink textGravatar kwzrd2019-12-09-1/+1
| | | |
| * | | Improve wording of cog docstringGravatar kwzrd2019-12-09-1/+1
| | | |
| * | | Log title wrapped in single quotes for readabilityGravatar kwzrd2019-12-09-1/+1
| | | |
| * | | Improve default title value and log messageGravatar kwzrd2019-12-09-1/+1
| | | | | | | | | | | | | | | | | | | | The title now defaults to Bookmark and is logged surrounded by single quotes to make it clear which part is the title.
| * | | Respond with a reaction on successGravatar kwzrd2019-12-09-3/+3
| | | | | | | | | | | | | | | | | | | | Instead of returning on error we add an else clause to the try-except block and add a reaction on success to reduce needless visual clutter.
| * | | Improve wording of log and response messagesGravatar kwzrd2019-12-09-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | Improves the wording of both docstrings, strips trailing dot from log and response messages for consistency. Refactors the error embed and renames the variable.
| * | | Fix indentationGravatar kwzrd2019-12-09-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | The bookmark method's function signature is reformatted, bookmark image url is reduced to a single line, and the embed's description is broken into separate lines.
| * | | lint test changeGravatar AG_14362019-12-09-2/+2
| | | |
| * | | K and R styling. replaced title with hintsGravatar AG_14362019-12-09-2/+3
| | | |
| * | | Changed how title behave when NoneGravatar AG_14362019-12-09-4/+1
| | | |
| * | | Removed footer , Unnecessary line. Edited a stringGravatar AG_14362019-12-09-3/+1
| | | |
| * | | Improved DM messageGravatar AG_14362019-12-08-7/+9
| | | | | | | | | | | | Simplified the DM , example in the original post
| * | | Reworked on the whole structure completely.Gravatar AG_14362019-12-08-35/+13
| | | | | | | | | | | | | | | | | | | | Removed error_embeds No more selecting last message if * or None is specified Redone how DM's Look like
| * | | Added better info, Added usage logging.Gravatar AG_14362019-12-07-1/+2
| | | |
| * | | renamed error completelyGravatar AG_14362019-12-05-2/+2
| | | |
| * | | Fixed typo of DMsGravatar AG_14362019-12-05-1/+1
| | | | | | | | | | | | Co-Authored-By: Matteo Bertucci <[email protected]>
| * | | Simple change in a if stringGravatar AG_14362019-12-05-1/+1
| | | | | | | | | | | | Co-Authored-By: Matteo Bertucci <[email protected]>
| * | | Renamed error namesGravatar AG_14362019-12-05-1/+1
| | | | | | | | | | | | Co-Authored-By: Matteo Bertucci <[email protected]>
| * | | Renamed error namesGravatar AG_14362019-12-05-1/+1
| | | | | | | | | | | | Co-Authored-By: Matteo Bertucci <[email protected]>
| * | | Saying goodbye to a eastereggGravatar AG_14362019-12-05-1/+0
| | | | | | | | | | | | RIP `embed.set_footer(text="Why everything so heavy ?")`
| * | | Fixed wrong order of import syntaxGravatar AG_14362019-12-05-2/+2
| | | |
| * | | Added embeds to error messages, new way of joining hints.Gravatar AG_14362019-12-05-11/+17
| | | |
| * | | Update bot/seasons/evergreen/bookmark.py Gravatar AG_14362019-12-05-1/+1
| | | | | | | | | | | | | | | | | | | | Fixed the description of the command itself Co-Authored-By: Matteo Bertucci <[email protected]>
| * | | Fixed cog string issueGravatar AG_14362019-12-05-11/+7
| | | | | | | | | | | | | | | | | | | | -Finally got a string that `describe` the cog Bookmark I love you Ves Zappa <3 . -Fixed no punctuation after `No hint provided` string. -Made it so it follow colo(u)ring theme of the bot.
| * | | Fixed a lot of issues.Gravatar AG_14362019-12-05-29/+24
| | | | | | | | | | | | | | | | | | | | Fixed the coding not satisfying `K&R` code styling Fixed most of the grammatical issues Removed unnecessary `try except` code block , not gonna trigger anyways.
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-27 01:21:09 +0000