aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Merge branch 'master' into feat/frontend/839/help-channel-roleGravatar Sebastiaan Zeeff2020-04-27-7/+50
|\
| * Merge pull request #904 from Akarys42/free-tagGravatar Shirayuki Nekomata2020-04-23-0/+5
| |\ | | | | | | Free tag
| | * Merge branch 'master' into free-tagGravatar Shirayuki Nekomata2020-04-23-12/+57
| | |\ | | |/ | |/|
| * | Merge pull request #900 from python-discord/fix-category-cache-issueGravatar Sebastiaan Zeeff2020-04-21-3/+2
| |\ \ | | | | | | | | Fix category cache issue
| | * | Fix category cache issueGravatar Joseph Banks2020-04-21-3/+2
| |/ /
| * | Merge pull request #896 from python-discord/answered-session-statsGravatar Mark2020-04-20-3/+42
| |\ \ | | | | | | | | Answered help session statistics
| | * | HelpChannels: add logging to answered checkGravatar MarkKoz2020-04-20-3/+4
| | | |
| | * | Extra documentation + split out to separate functionGravatar Joseph Banks2020-04-20-8/+26
| | | |
| | * | Add different emoji for different channel statuses (in use answered/unanswered)Gravatar Joseph Banks2020-04-20-2/+7
| | | |
| | * | Incorrect comparison, we need to check if we are in help_in_use, not out of itGravatar Joseph Banks2020-04-20-1/+1
| | | |
| | * | Add statistics on whether a help session was closed with no input from ↵Gravatar Joseph Banks2020-04-20-0/+15
| |/ / | | | | | | | | | anyone but the claimant
| * | Merge pull request #875 from python-discord/bug/info/869/tag-encodingGravatar Mark2020-04-19-1/+1
| |\ \ | | | | | | | | Explicitly use UTF-8 to read tag files
| | * \ Merge branch 'master' into bug/info/869/tag-encodingGravatar Mark2020-04-19-52/+354
| | |\ \ | | |/ / | |/| |
| | * | Tags: explicitly use UTF-8 to read filesGravatar MarkKoz2020-04-10-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Not all operating systems use UTF-8 as the default encoding. For systems that don't, reading tag files with Unicode would cause an unhandled exception.
| | | * Free tag: fix typo in headerGravatar kwzrd2020-04-23-1/+1
| | | |
| | | * Simplify free tagGravatar Matteo Bertucci2020-04-23-3/+2
| | | |
| | | * Use IDs instead of hard-coding category names in the free tagGravatar Matteo Bertucci2020-04-16-2/+1
| | | | | | | | | | | | Co-Authored-By: Shirayuki Nekomata <[email protected]>
| | | * Add a title at the top of the free tagGravatar Matteo Bertucci2020-04-16-0/+2
| | | | | | | | | | | | Co-Authored-By: Shirayuki Nekomata <[email protected]>
| | | * Create temporary free tagGravatar Matteo Bertucci2020-04-16-0/+5
| | | |
* | | | HelpChannels: rename dormant command to closeGravatar MarkKoz2020-04-20-4/+4
| | | | | | | | | | | | | | | | | | | | People are more familiar with the "close" alias than its actual name, "dormant". "close" also feels more natural.
* | | | HelpChannels: check if the help cooldown role existsGravatar MarkKoz2020-04-20-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | A NotFound error can be misleading since it may apply to the member or the role. The log message was not simply updated because each of the scenarios need to have different log levels: missing members is a normal thing but an invalid role is not.
* | | | HelpChannels: pass coroutine func instead to `_change_cooldown_role`Gravatar MarkKoz2020-04-20-6/+8
| | | | | | | | | | | | | | | | | | | | This will allow `_change_cooldown_role` to handle the role argument rather than putting that burden on the callers.
* | | | HelpChannels: tidy up log messagesGravatar MarkKoz2020-04-20-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | * Remove obsolete log message * Shorten a log message which was the only line in the entire module over 100 characters
* | | | HelpChannels: handle errors when changing cooldown roleGravatar MarkKoz2020-04-17-12/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A user may leave the guild before their role can be changed. Sometimes, there could also be role hierarchy issues or other network issues. It's not productive to halt everything and just dump these as exceptions to the loggers. The error handler provides a more graceful approach to these exceptions. * Add a wrapper function around `add_roles` & `remove_roles` which catches exceptions
* | | | HelpChannels: fix role not resetting after dormant commandGravatar MarkKoz2020-04-17-13/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Resetting permissions relied on getting the member from the cache, but the member was already removed from the cache prior to resetting the role. Now the member is passed directly rather than relying on the cache.
* | | | HelpChannels: mention dormant cmd in available message embedGravatar MarkKoz2020-04-17-2/+3
| | | | | | | | | | | | | | | | Users should know they can close their own channels.
* | | | HelpChannels: add info about cooldown role & dormant cmd to docstringGravatar MarkKoz2020-04-17-0/+3
| | | |
* | | | HelpChannels: add/remove a cooldown role rather than using overwritesGravatar MarkKoz2020-04-17-13/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Overwrites had issues syncing with channels in the category. * Remove update_category_permissions; obsolete * Add constant for the cooldown role wrapped in a discord.Object
* | | | HelpChannels: clear roles when resetting permissionsGravatar MarkKoz2020-04-17-9/+7
| | | | | | | | | | | | | | | | | | | | Claimants will have a special role that needs to be removed rather than using member overwrites for the category.
* | | | HelpChannels: add helper function to check for claimant roleGravatar MarkKoz2020-04-17-0/+5
| | | |
* | | | HelpChannels: remove ensure_permissions_synchronizationGravatar MarkKoz2020-04-17-29/+1
| | | |
* | | | Constants: add help channel cooldown roleGravatar MarkKoz2020-04-17-0/+2
|/ / /
* | | Merge pull request #877 from ↵Gravatar Shirayuki Nekomata2020-04-18-2/+8
|\ \ \ | | | | | | | | | | | | | | | | python-discord/bug/frontend/870/help-channel-dm-category HelpChannels: fix AttributeError getting a category for a DMChannel
| * \ \ Merge branch 'master' into bug/frontend/870/help-channel-dm-categoryGravatar Shirayuki Nekomata2020-04-18-50/+346
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #874 from ks129/syncer-timeout-fixGravatar kwzrd2020-04-17-2/+4
|\ \ \ \ | | | | | | | | | | Fix wrong exception type in syncing
| * \ \ \ Merge branch 'master' into syncer-timeout-fixGravatar kwzrd2020-04-17-181/+1253
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Remove `.md` from anti-malware whitelistGravatar Sebastiaan Zeeff2020-04-16-1/+0
| |_|_|/ |/| | | | | | | We want our members to use the paste site to share text-based files instead of them sharing the files as attachments on Discord. As `.md`, a file extensions used for plain-text files with markdown formatting, is such a text file, I've removed it from the anti-malware whitelist.
* | | | Merge pull request #881 from ↵Gravatar Shirayuki Nekomata2020-04-14-4/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/help-channels-keep-dormant-invocation Reverse deletion of `!dormant` invocation messages
| * | | | Revert deletion of !dormant invocation messagesGravatar Sebastiaan Zeeff2020-04-14-4/+0
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR #868 introduced the automatic deletion of the message that issued the `!dormant` command. The idea behind this was that moving the channel to the dormant category makes it obvious that a channel has gone dormant and the message would only serve as visual clutter. However, removing the command invocation also means that it's less obvious why a channel was moved to the dormant category. As the message gets deleted almost immediately, you have to be actively watching the channel to know that the command was issued and who issued it. This has already caused some confusion where helping members where left wondering why a channel suddenly went dormant while they felt that the conversation was still ongoing. To improve the user experience, this commit removes the deletions of the command invocation messages.
* | | | [stat] Tag statistic was using the user input as the series name, not the ↵Gravatar Joseph Banks2020-04-13-1/+1
| | | | | | | | | | | | | | | | resolved tag name
* | | | [stat] Create a statistic for whether dormant was called by the claimant or ↵Gravatar Joseph Banks2020-04-12-1/+7
| | | | | | | | | | | | | | | | staff
* | | | Merge branch 'Numerlor-help-dormant-feedback' into `master`Gravatar Sebastiaan Zeeff2020-04-12-10/+53
|\ \ \ \ | | | | | | | | | | | | | | | After resolving the merge conflicts created by the recent stats addition, I'm merging this into master. No functional changes were made during conflict resolution (hopefully) and the merge has been tested.
| * | | | Merge branch 'help-dormant-feedback' of https://github.com/Numerlor/bot into ↵Gravatar Sebastiaan Zeeff2020-04-12-10/+53
|/| | | | | | | | | | | | | | | | | | | Numerlor-help-dormant-feedback
| * | | | Fix `help_channel_claimants` typehint.Gravatar Numerlor2020-04-10-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `ctx.author` that is used to populate the dict returns a `Member` object in most cases while only `User` was documented as a possible value.
| * | | | Use synchronized permission reset.Gravatar Numerlor2020-04-10-1/+1
| | | | |
| * | | | Specify encoding to logging file handler.Gravatar Numerlor2020-04-09-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the new addition of non latin-11 chars in channel names - which get logged, the logging to files fails on those entries on OSs where the default encoding is not utf8 or an other encoding capable of handling them.
| * | | | Delete channel from claimant cache.Gravatar Numerlor2020-04-09-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deleting the channel from the claimant cache on invokation of the dormant command prevents users running the command multiple times before the bot moves it.
| * | | | Reverse order of moving to dormant and task cancellation.Gravatar Numerlor2020-04-09-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reversing the order ensures the task is not cancelled when moving to dormant fails which gives a fallback to move it after the initial period of time.
| * | | | HelpChannels: check author of dormant messageGravatar MarkKoz2020-04-09-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In a testing environment, the bot may try to edit the message of a different bot. Therefore, the author of the message should be checked to ensure the current bot sent it.
| * | | | Suppress errors when resetting permissions.Gravatar Numerlor2020-04-09-1/+2
| | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-30 04:57:31 +0000