aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | 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
|/ / /
| * | Update bot/seasons/evergreen/trivia_quiz.pyGravatar Rohan_Iceman2019-12-20-1/+1
| | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | Update bot/seasons/evergreen/trivia_quiz.pyGravatar Rohan_Iceman2019-12-20-1/+3
| | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | fixed lint errorsGravatar RohanJnr2019-12-20-1/+1
| | |
| * | Merge branch 'quiz_fix' of https://github.com/RohanJnr/seasonalbot into quiz_fixGravatar RohanJnr2019-12-20-6/+18
| |\ \
| | * | Trivia: fix spelling error in category embed footerGravatar MarkKoz2019-12-17-1/+1
| | | |
| | * | Trivia: fix concatenation of winner mentionsGravatar MarkKoz2019-12-17-3/+1
| | | |
| | * | Trivia: space out some of lines of codeGravatar MarkKoz2019-12-17-3/+17
| | | |
| * | | In this commit:Gravatar RohanJnr2019-12-20-12/+13
| |/ / | | | | | | | | | | | | | | | | | | - I cleaned up some residue code - The quiz now selects a random category if None provided - Updated doc strings - Displaying the category when the quiz is starting
| * | fixed a indent errorGravatar RohanJnr2019-12-16-7/+6
| | |
| * | Merge branch 'quiz_fix' of https://github.com/RohanJnr/seasonalbot into quiz_fixGravatar RohanJnr2019-12-16-1/+1
| |\ \
| | * | Update bot/seasons/evergreen/trivia_quiz.py Gravatar Rohan_Iceman2019-12-02-1/+1
| | | | | | | | | | | | | | | | | | | | adding space around special chars Co-Authored-By: Mark <[email protected]>
| * | | Fixed indentation, the category_embed function is no longer a property.Gravatar RohanJnr2019-12-16-28/+22
| |/ /
| * | added a new command to stop the quiz and removed the cooldown on the quiz ↵Gravatar RohanJnr2019-11-29-6/+15
| | | | | | | | | | | | command
| * | Merge branch 'master' into quiz_fixGravatar Rohan_Iceman2019-11-28-134/+722
| |\ \
| * | | Grammar fixesGravatar Joseph Banks2019-11-22-19/+16
| | | |
| * | | added cooldown to the quiz command and more refinement and also tested.Gravatar RohanJnr2019-10-20-1/+2
| | | |
| * | | added a cool down per channel to the quiz commandGravatar RohanJnr2019-10-19-2/+6
| | | |
| * | | Made some language corrections in the json file and also made seperate ↵Gravatar RohanJnr2019-10-18-23/+39
| | | | | | | | | | | | | | | | functions for the start and end quiz. Added another check to check if the game is still running before sending the answer.
| * | | fixed bugs such as incorrect time to send the category embed, game will no ↵Gravatar RohanJnr2019-10-17-16/+18
| | | | | | | | | | | | | | | | longer run again if stopped n stuff.
| * | | Updated the code as per the suggestions and they are the following:Gravatar RohanJnr2019-10-13-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | - Corrected all typos. - the quiz command is now a command group to allocate the leaderboard command(a command which shows the leaderboard for the current bot session). - Few other refinements with language and stuff.
| * | | fixed lint errorsGravatar RohanJnr2019-10-07-1/+1
| | | |
| * | | added function annotations to new commandGravatar RohanJnr2019-10-07-1/+1
| | | |
| * | | 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.