aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | Merge pull request #2342 from python-discord/don't-close-already-closed-postsGravatar ChrisJL2022-11-27-0/+6
|\ \ | | | | | | Don't close already closed posts
| * | Don't close already closed postsGravatar Chris Lovering2022-11-27-0/+6
|/ /
* | Merge pull request #2333 from erlend-aasland/patch-1Gravatar wookie1842022-11-27-1/+1
|\ \ | | | | | | Link directly to the sqlite3 placeholder howto
| * \ Merge branch 'main' into patch-1Gravatar wookie1842022-11-27-1970/+1034
| |\ \ | |/ / |/| |
* | | Merge pull request #2339 from python-discord/auto-archive-help-postsGravatar Boris Muratov2022-11-26-64/+165
|\ \ \ | | | | | | | | Help forum improvements
| * | | Catch case where starter message is deleted before pinningGravatar Chris Lovering2022-11-26-2/+4
| | | |
| * | | Update CODEOWNERSGravatar Chris Lovering2022-11-26-1/+0
| | | |
| * | | Ensure constants.Channels.help_system_forum is a forum channel on loadGravatar Chris Lovering2022-11-26-1/+3
| | | |
| * | | Listen for thread starter on_message rather than thread_createGravatar Chris Lovering2022-11-26-10/+19
| | | | | | | | | | | | | | | | thread_create events are triggered before the thread's starter message is available, so listening for starter messages with on_message instead ensures we have a message object to use.
| * | | Auto archive help forum posts after inactivityGravatar Chris Lovering2022-11-26-1/+78
| | | | | | | | | | | | | | | | | | | | | | | | I decided to keep the archive logic simple, and just go for 30 minutes since last message, rather than the hybrid of 30 mins + 10 depending on who sent the last message. The reason for using the hybrid approach previously was due to us running out of channels frequently Since this is no longer a problem, I decided to keep the logic simple.
| * | | Don't load the help channel cog if disabledGravatar Chris Lovering2022-11-26-0/+7
| | | |
| * | | Consistantly refer to help posts as posts, not threadsGravatar Chris Lovering2022-11-26-47/+47
| | | |
| * | | Get, and store, the help forum channel object on cog loadGravatar Chris Lovering2022-11-26-4/+9
|/ / /
| * | Link directly to the sqlite3 placeholder howtoGravatar Erlend E. Aasland2022-11-25-1/+1
| | |
| | * make the roles view ephemeral when sent in roles channelGravatar shtlrs2022-11-27-1/+2
| | |
| | * add docstrings to RoleButtonViewGravatar shtlrs2022-11-27-6/+17
| | | | | | | | | | | | This documents the attributes the class has, which gives better context for future readers/contributors
| | * misc style & doc improvementsGravatar shtlrs2022-11-27-3/+3
| | |
| | * remove waring in setup docstringsGravatar shtlrs2022-11-27-1/+1
| | |
| | * rename the method that attaches the persistent viewGravatar shtlrs2022-11-27-4/+12
| | | | | | | | | | | | This also enhances the docstrings for a better clarification of its purpose
| | * update docs for the newly added view & buttonGravatar shtlrs2022-11-27-2/+2
| | |
| | * rename prepare_available_role_subscription_view to ↵Gravatar shtlrs2022-11-27-3/+3
| | | | | | | | | | | | prepare_self_assignable_roles_view
| | * fix docstrings of the ShowAllSelfAssignableRolesButton's callback functionGravatar shtlrs2022-11-27-1/+1
| | |
| | * update message content of the persistent viewGravatar shtlrs2022-11-27-2/+2
| | | | | | | | | | | | | | | The messages were intended for a button that will automatically assign all availables roles to the interactor This changes since it's not the intended behavior
| | * rename ClaimAllSelfAssignableRolesButton to ShowAllSelfAssignableRolesButtonGravatar shtlrs2022-11-27-5/+5
| | |
| | * restore original value of DELETE_MESSAGE_AFTERGravatar shtlrs2022-11-27-1/+1
| | |
| | * rely on original_message to delete view once it times outGravatar shtlrs2022-11-27-66/+62
| | |
| | * add implementation of the button's callback that'll handle assigning the ↵Gravatar shtlrs2022-11-26-0/+34
| | | | | | | | | | | | appropriate roles
| | * add assignable_roles as a property to the ClaimAllSelfAssignableRoles buttonGravatar shtlrs2022-11-26-3/+4
| | |
| | * add the logic for attaching the persistent viewGravatar shtlrs2022-11-26-1/+36
| | | | | | | | | | | | | | | | | | This comes with another dependency to try to locate the message that needs to hold the view. If that message is not found, a new one will be created & sent.
| | * add the AllSelfAssignableRolesView and its corresponding ↵Gravatar shtlrs2022-11-26-1/+21
| | | | | | | | | | | | | | | | | | ClaimAllSelfAssignableRolesButton button Note that these are still dummy views & have no behavior
| | * add the roles channel to the configGravatar shtlrs2022-11-26-0/+5
| |/ |/|
* | Merge pull request #2338 from python-discord/bump-bot-coreGravatar ChrisJL2022-11-26-690/+595
|\ \ | | | | | | Bump bot core version
| * | Ignore Discord errors when clearing snekbox reactionsGravatar Chris Lovering2022-11-26-2/+3
| | | | | | | | | | | | These errors can be caused by the message being deleted, or the message being in an archived thread
| * | Update botcore module nameGravatar Chris Lovering2022-11-26-59/+59
| | |
| * | Bump all deps to latestGravatar Chris Lovering2022-11-26-403/+303
| | |
| * | Bump bot-core versionGravatar Chris Lovering2022-11-26-232/+236
|/ /
* | 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
| |