aboutsummaryrefslogtreecommitdiffstats
path: root/bot/exts (follow)
Commit message (Collapse)AuthorAgeLines
* Decrease timeout from 120 sec to 30 secGravatar ks1292021-01-24-1/+1
|
* Fix indentionGravatar ks1292021-01-17-1/+1
|
* Remove unnecessary line splitGravatar ks1292021-01-17-3/+1
| | | Co-authored-by: Xithrius <[email protected]>
* Merge branch 'master' into tic-tac-toeGravatar ks1292021-01-16-1/+1
|\
| * Actually use 256 coloursGravatar Chris2021-01-09-1/+1
| |
* | Fix too long lineGravatar ks1292021-01-16-1/+4
| |
* | More grammar fixesGravatar ks1292021-01-16-2/+2
| |
* | Improve "your turn" messageGravatar ks1292021-01-16-1/+1
| |
* | Add missing a articleGravatar ks1292021-01-16-1/+1
| | | | | | Co-authored-by: ChrisJL <[email protected]>
* | Fix grammarGravatar ks1292021-01-16-1/+1
| | | | | | Co-authored-by: ChrisJL <[email protected]>
* | Move Tic Tac Toe board to embed descriptionGravatar ks1292021-01-16-2/+6
| |
* | Simplify check_win function returningGravatar ks1292021-01-09-12/+12
| |
* | Corrected small spelling mistake.Gravatar xithrius2021-01-08-4/+4
| |
* | Merge remote-tracking branch 'origin/master' into tic-tac-toeGravatar xithrius2021-01-08-67/+79
|\|
| * Merge branch 'master' into hackstatsGravatar ks1292021-01-06-852/+1033
| |\
| | * Remove unused importGravatar William Da Silva2020-12-29-1/+1
| | |
| | * Merge branch 'master' into omdb-to-tmdbGravatar William Da Silva2020-12-28-800/+982
| | |\
| | * \ Merge branch 'master' into omdb-to-tmdbGravatar Matteo Bertucci2020-11-30-950/+605
| | |\ \
| | * | | Fix empty field in snake movie embedGravatar Will Da Silva2020-10-16-2/+4
| | | | |
| | * | | Comply with TMDB ToSGravatar Will Da Silva2020-10-16-1/+2
| | | | |
| | * | | Set the number of movie pages fetched on first requestGravatar Will Da Silva2020-10-13-1/+5
| | | | |
| | * | | Fix TMDB leakGravatar Will Da Silva2020-10-08-1/+1
| | | | |
| | * | | Replace OMDB with TMDBGravatar Will Da Silva2020-10-05-51/+43
| | | | | | | | | | | | | | | | | | | | Closes: #136
| * | | | Formatting and add full stop to docstringGravatar Hedy Li2021-01-06-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | - bot/exts/halloween/hacktoberstats.py line 130, better readability - same file line 208-209 add full stop
| * | | | Remove unused pathlib.Path importGravatar Hedy Li2020-12-03-1/+0
| | | | |
| * | | | Merge branch 'master' into hackstatsGravatar Matteo Bertucci2020-11-30-858/+243
| |\ \ \ \ | | | |/ / | | |/| |
| * | | | Continue loop if repo topics API request erroredGravatar Hedy Li2020-10-31-1/+1
| | | | |
| * | | | Fix capitalization of 'PR' in hacktoberstats.pyGravatar Hedy Li2020-10-31-1/+1
| | | | |
| * | | | Put GitHub user-not-found message in embedGravatar Hedy Li2020-10-31-2/+5
| | | | | | | | | | | | | | | | | | | | With random `NEGATIVE_REPLIES` + color=red
| * | | | Fix capitalization and grammarGravatar Hedy Li2020-10-31-12/+12
| | | | | | | | | | | | | | | For Hacktoberstats
| * | | | better no-prs messageGravatar Hedy Li2020-10-22-1/+1
| | | | |
| * | | | improve implementation with return valueGravatar Hedy Li2020-10-22-12/+11
| | | | | | | | | | | | | | | | | | | | instead of string/None now its empty list/None
| * | | | Merge branch 'master' of https://github.com/python-discord/seasonalbot into ↵Gravatar Hedy Li2020-10-22-1/+1
| |\ \ \ \ | | | | | | | | | | | | | | | | | | hackstats
| * | | | | improve message when user not foundGravatar Hedy Li2020-10-22-7/+14
| | | | | |
* | | | | | Merge branch 'master' into tic-tac-toeGravatar ks1292020-12-28-801/+983
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | |
| * | | | | Merge branch 'master' into prideavatar-urlGravatar ks1292020-12-28-764/+913
| |\ \ \ \ \
| | * | | | | Clarify comment on AoC Status Task startup delayGravatar Sebastiaan Zeeff2020-12-13-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Advent of Code Status Countdown task needs to wait for two things to happen to prevent it from failing during the startup sequence: 1. The Websocket instance discord.py creates needs to be available as an attribute of the bot, otherwise discord.py fails internally: Traceback (most recent call last): File "discord/client.py", line 1049, in change_presence await self.ws.change_presence( activity=activity, status=status, afk=afk ) File "advent_of_code/_cog.py", line 52, in countdown_status await bot.change_presence(activity=discord.Game(playing)) AttributeError: 'NoneType' object has no attribute 'change_presence' 2. Allegedly, according to the discord.py community, trying to change the status too early in the sequence to establish a connection with Discord may result ub the Discord API aborting the connection. To solve this, I've added a `wait_until_guild_available` waiter, as it guarantees that the websocket is available and the connections is mature. Kaizen: I've changed the name `new_puzzle_announcement` to `new_puzzle_notification` to better reflect its function.
| | * | | | | Merge branch 'master' into sebastiaan/advent-of-code/refactor-background-tasksGravatar Sebastiaan Zeeff2020-12-13-58/+133
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # bot/exts/christmas/advent_of_code/_cog.py # bot/exts/christmas/advent_of_code/_helpers.py
| | | * | | | | Add callback to log errors in AoC background tasksGravatar Sebastiaan Zeeff2020-12-13-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, our Advent of Code background tasks fail without logging errors or printing error messages. This makes it difficult to debug the errors and means that they may fail silently. While we should ideally find the root cause that hides such errors, I've added a done_callback function in the meantime to help us debug the current issues with the Advent of Code Notification Task.
| | | * | | | | Change AOC_WHITELIST names for clarityGravatar janine9vn2020-12-11-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AOC_WHITELIST was changed to AOC_WHITELIST_RESTRICTED because it is clearer that commands with this parameter in the `@override_in_channel()` decorator will be restricted to the aoc commands channel and not be allowed in the main aoc channel. In the same vein, AOC_WHITELIST_PLUS was changed to AOC_WHITELIST.
| | | * | | | | Remove extra pleaseGravatar janine9vn2020-12-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Please -= 1
| | | * | | | | Remove re-raising the errorGravatar janine9vn2020-12-11-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per Mark's comment, re-raising the error isn't necessary.
| | | * | | | | Change custom error handler to match new styleGravatar janine9vn2020-12-11-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm a bit ahead of the game and changing the error handler to match the new style that Iceman will PR shortly.
| | | * | | | | Add cog-level error handler for Incorrect ChannelGravatar janine9vn2020-12-11-6/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If any of the "spammier" commands (stats, leaderboard) are used within the primary advent of code channel, rather than a non-specific embed we instead reply with the channel they should be using. This also adds a "AOC_WHITELIST_PLUS" constant that makes it easier to adjust what channels the non-spammier aoc commands can be used in.
| | | * | | | | Disallow .aoc commands in primary aoc channelGravatar janine9vn2020-12-11-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commands like `.aoc leaderboard` and `.aoc stats` proved to be spammy in the main advent of code channel. An aoc_commands channel has been added for aoc commands and this update prohibits aoc commands from being used in the primary aoc channel and adds the comands channel to the whitelist. This also specifically allows the less spammier commands: join, subscribe, unsubscribe, and countdown in the primary channel to foster discussion though.
| | | * | | | | Merge pull request #540 from RohanJnr/error_handler_changesGravatar Mark2020-12-11-21/+10
| | | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | Modify error handler check for locally handled errors.
| | | | * | | | | Check if error.original is an instance of OSError.Gravatar Rohan2020-12-11-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, remove error handler for get_command and video_command.
| | | | * | | | | Modify snakes_cog error handler.Gravatar Rohan2020-12-07-16/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | local error handler no longer checks for BadArgument error and the attribute handled will be set to True on the error if an OSError occurs.
| | | | * | | | | Check value of handled attribute on error in global error handler.Gravatar Rohan2020-12-04-1/+1
| | | | | | | | |
| | | | * | | | | Modify error handler check for locally handled errors.Gravatar Rohan2020-12-02-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error handler now checks if the error has the attribute "handled" for locally handled errors.