aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge pull request #2337 from python-discord/help-forum-fixesGravatar ChrisJL2022-11-26-64/+76
|\ | | | | Misc Fixes For Help Forum
| * Refactor Thread Error CatchingGravatar Hassan Abouelela2022-11-26-17/+17
| | | | | | | | | | | | Co-authored-by: Boris Muratov <[email protected]> Co-authored-by: Amrou Bellalouna <[email protected]> Signed-off-by: Hassan Abouelela <[email protected]>
| * Handle Images As Starter MessagesGravatar Hassan Abouelela2022-11-26-1/+5
| | | | | | | | | | | | | | | | In the case of an image or other media as the starter message, the formatted message in the help DM will be empty, which is invalid for the embed. We populate the field with some more useful text in this case. Signed-off-by: Hassan Abouelela <[email protected]>
| * Check If Thread Is Closed In PaginationGravatar Hassan Abouelela2022-11-26-55/+38
| | | | | | | | | | | | | | Similar to 555ed4e9, the pagination utility needs to catch when it's trying to act on an archived thread. Signed-off-by: Hassan Abouelela <[email protected]>
| * Check If Thread Is Closed In wait_for_deletionGravatar Hassan Abouelela2022-11-26-1/+8
| | | | | | | | | | | | | | | | | | | | | | The wait_for_deletion utility would try to remove reactions from a message after the timeout expires, which would normally be fine. In threads however, they can be closed while waiting for the timeout to expire. In such a case, the bot will try to remove the reactions after the channel has been closed and fail. A special exception was added for this case to do nothing, since this is only a QoL feature. Signed-off-by: Hassan Abouelela <[email protected]>
| * Catch Failure In Pining Help Starter MessageGravatar Hassan Abouelela2022-11-26-2/+7
| | | | | | | | | | | | | | | | | | The old method for detecting deleted opener messages does not seem to work, probably because the message is fetched from a cache or similar. Instead we simply try/except pinning the message and pass if the pinning failed. Signed-off-by: Hassan Abouelela <[email protected]>
| * Don't Remove Cooldown Role From Non-Existing UsersGravatar Hassan Abouelela2022-11-26-0/+9
| | | | | | | | | | | | | | | | We try to remove the cooldown role from users before checking if the user is still in the server, which can cause an error since the thread object will just contain `None` as the user. Signed-off-by: Hassan Abouelela <[email protected]>
| * Handle Discord API Not Being ReadyGravatar Hassan Abouelela2022-11-26-3/+7
| | | | | | | | | | | | | | | | | | | | When discord sends us the thread create event in help channels, it is not ready to perform other operations on the thread such as getting or pinning messages. This causes it to error out when we try to do these actions and claim that those channels don't exist. Instead, we sleep for a short time to try and wait for it to be ready. Signed-off-by: Hassan Abouelela <[email protected]>
| * Simplify Help Thread Warning LogGravatar Hassan Abouelela2022-11-26-1/+1
|/ | | | | | | | | The current warning log includes the thread name, which means the log message varies wildly between threads. This causes issues with sentry since the actual error message gets trimmed, and sentry fails to group issues from this log as they appear as different messages. Signed-off-by: Hassan Abouelela <[email protected]>
* Merge pull request #2335 from python-discord/help-system-timeoutGravatar Boris Muratov2022-11-26-1/+2
|\ | | | | Send DM message to help post opener sooner in the process
| * Send DM message to help post opener sooner in the processGravatar Chris Lovering2022-11-25-1/+2
|/ | | | | | This is to give Discord time to make the thread actually avaiulable to be posted in for the send_opened_post_message call Closes #2334 Closes BOT-3AW
* Merge pull request #2318 from python-discord/help-channel-rewriteGravatar ChrisJL2022-11-25-1256/+301
|\ | | | | Update help channel system to use forum channels
| * Pin the user's starter message on help post creationGravatar Chris Lovering2022-11-25-0/+4
| |
| * Add hf as an alias for the help-forum command groupGravatar Chris Lovering2022-11-25-1/+1
| |
| * Add a filter for help chanel post namesGravatar Chris Lovering2022-11-25-2/+25
| | | | | | | | It is expected that this code will be delete whent he new fitler cog is added, and we start filtering on thread names genericly.
| * Use a redis cache to determine if a help session was answeredGravatar Chris Lovering2022-11-25-4/+8
| |
| * Remove modlog ignore for help chanelsGravatar Chris Lovering2022-11-25-7/+1
| | | | | | | | This was due to the hlep channels causing many events top be pushed to modlog due to how the old system worked. Now that we use a forum chanel, this is no longer the case.
| * Use helper util to determine if snekbox is being invoked in a help postGravatar Chris Lovering2022-11-25-2/+3
| |
| * Use help channel util from help channels cogGravatar Chris Lovering2022-11-25-11/+2
| | | | | | | | This removes the need for the old helper entirely
| * Update help channel system to use forum channelsGravatar Chris Lovering2022-11-25-1162/+263
| | | | | | | | This leverages Discord's new forum chanel feature, which removes the need for a lot of our custom logic, simplifying the help channel cog significantly.
| * Remove unused cachesGravatar Chris Lovering2022-11-25-21/+0
| |
| * Update constants for new forum channel help systemGravatar Chris Lovering2022-11-25-56/+4
|/
* Edited several tags (#2322)Gravatar Keyacom2022-11-17-19/+34
| | | * Edited several tags
* Merge pull request #2320 from thegamecracks/intents-tagGravatar wookie1842022-11-12-7/+7
|\ | | | | Update intents tag to better reflect discord.py's API changes
| * Merge branch 'main' into intents-tagGravatar thegamecracks2022-11-12-5/+5
| |\ | |/ |/|
* | Merge pull request #2321 from thegamecracks/discordpy-stable-docsGravatar wookie1842022-11-10-5/+5
|\ \ | | | | | | Update discord.py tags to refer to the stable version of the documentation
| * | Fix reference link in customcooldown.mdGravatar thegamecracks2022-11-10-1/+1
| | |
| * | Change discord.py links in tags to use stable versionGravatar thegamecracks2022-11-10-5/+5
|/ / | | | | | | | | | | The `latest` version of docs follows discord.py's development branch, which may include changes to documentation that are not relevant/accessible to typical users of the library.
| * Add further clarification to intents.mdGravatar thegamecracks2022-11-10-3/+5
| | | | | | | | | | | | | | - References discord.Intents documentation for detail on available intents and which features they impact - Explicitly describes no requirements for standard intents - Adds a code comment explaining the intents being enabled
| * Update intents.mdGravatar thegamecracks2022-11-10-6/+4
|/ | | | | | | | - More generally describes features enabled by intents - Adds a disclaimer for discord.py v2.0.0 requiring intents - Adds distinction between standard and privileged intents - Replaces `members` intent in code snippet with `message_content` in accordance with the usage of `commands.Bot`
* Merge pull request #2307 from python-discord/2306-nominations-cleanupGravatar wookie1842022-11-05-279/+323
|\ | | | | Talentpool Cleanup
| * Merge branch 'main' into 2306-nominations-cleanupGravatar wookie1842022-11-05-27/+86
| |\ | |/ |/|
* | Merge pull request #2230 from meatballs/create-nomodule-tagGravatar ChrisJL2022-11-02-0/+13
|\ \
| * \ Merge branch 'main' into create-nomodule-tagGravatar ChrisJL2022-11-02-1494/+2457
| |\ \ | |/ / |/| |
* | | Merge pull request #2309 from shtlrs/issue-2280-rule-keywords-discoverabilityGravatar ChrisJL2022-11-02-0/+15
|\ \ \
| * \ \ Merge branch 'main' into issue-2280-rule-keywords-discoverabilityGravatar ChrisJL2022-11-02-27/+58
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #2315 from python-discord/bug/2299/timeitGravatar ChrisJL2022-11-02-5/+10
|\ \ \ \
| * \ \ \ Merge branch 'main' into bug/2299/timeitGravatar ChrisJL2022-11-02-1/+25
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #2285 from Keyacom/Keyacom-patch-1Gravatar ChrisJL2022-11-02-0/+24
|\ \ \ \ \
| * \ \ \ \ Merge branch 'main' into Keyacom-patch-1Gravatar ChrisJL2022-11-02-362/+1962
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #2311 from python-discord/mbaruh-bump-psqlGravatar wookie1842022-10-30-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Bump PSQL version in docker-compose
| * | | | | | Bump PSQL version in docker-composeGravatar mbaruh2022-10-28-1/+1
|/ / / / / /
| * | | | | Simplify REPL exampleGravatar Keyacom2022-10-31-2/+0
| | | | | |
| * | | | | Update slicing.mdGravatar Keyacom2022-10-30-22/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Did things according to what was proposed by @swfarnsworth, and the following: - Fixed the REPL example (assignments never return) - The word "slicing" at the start was not capitalized in the proposal, even though it started a sentence. I fixed that.
| * | | | | added file to another branchGravatar Keyacom2022-10-04-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added file to another branch because it could interfere with the `main` branch when a pull request is made. This file adds an explanation on sequence slicing.
| | * | | | Pass args to the Python version switch buttonGravatar Karlis Suvi2022-10-28-5/+10
| |/ / / / |/| | | |
* | | | | Merge pull request #2310 from python-discord/rules-fixGravatar ChrisJL2022-10-27-20/+22
|\ \ \ \ \ | | | | | | | | | | | | Fix for rules greedy parsing freezing
| * | | | | Reverse changes to invalid arg breakGravatar Ionite2022-10-25-5/+8
| | | | | |
| * | | | | Update tests to work with rules arg fixGravatar Ionite2022-10-25-4/+3
| | | | | |
| * | | | | Add args split max limitGravatar Ionite2022-10-25-1/+1
| | | | | |