aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' into reddit_postsGravatar Rohan2020-02-05-5/+16
|\
| * Update CODEOWNERSGravatar Joseph2020-02-02-1/+1
| |
| * Create CODEOWNERSGravatar Joseph2020-02-02-0/+1
| |
| * Merge pull request #340 from AG4lyf/bookmark-staff-fixGravatar Shirayuki Nekomata2020-01-28-1/+13
| |\ | | | | | | Added permission check for staff channels message leak.
| | * Shorten description & remove author override in permission error embedGravatar S. Co12020-01-27-4/+4
| | | | | | | | | Update logging events so they follow the correct chain of execution
| | * Added permission check for staff channels message leak.Gravatar AG_14362020-01-25-0/+12
| |/
| * Merge pull request #337 from AG4lyf/bookmark-patchGravatar Shirayuki Nekomata2020-01-09-4/+2
| |\ | | | | | | Add exception where relayed message.content length exceed 2048 charac…
| | * Removed unnacessary `content` variable usageGravatar AG_14362020-01-07-2/+1
| | |
| | * Moved jump url into a field.Gravatar AG_14362020-01-07-6/+2
| | |
| | * Add exception where relayed message.content length exceed 2048 charactersGravatar AG_14362020-01-07-1/+4
| |/
* | uninstalled the buttons lib and corrected lint errorsGravatar RohanJnr2020-02-05-243/+170
| |
* | finished implementing the reddit commandGravatar RohanJnr2020-02-05-45/+51
| |
* | installed buttons and implemented embed paginationGravatar RohanJnr2020-01-26-157/+321
|/
* Use a static discord shield on the readme.Gravatar scragly2019-12-26-2/+2
|
* Merge pull request #298 from RohanJnr/quiz_fixGravatar Mark2019-12-20-77/+147
|\ | | | | Quiz command fix
| * Merge branch 'master' into quiz_fixGravatar Mark2019-12-20-211/+430
| |\ | |/ |/|
* | Merge pull request #332 from python-discord/errorhandler-refineGravatar Shirayuki Nekomata2019-12-16-119/+106
|\ \ | | | | | | Use error_embed for errors, tidy code, show cmd signature for user errors.
| * | Auto-delete channel check error after a period of time.Gravatar scragly2019-12-12-2/+2
| | |
| * | Add exc_info to log.exception to display traceback.Gravatar scragly2019-12-12-1/+1
| | |
| * | Use error_embed for errors, tidy code, show cmd signature for user errors.Gravatar scragly2019-12-12-119/+106
|/ /
* | 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 ?")`
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-26 17:24:25 +0000