aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Update `patreon_info` function docstringGravatar mathstrains212021-11-28-1/+1
| | | Co-authored-by: Bluenix <[email protected]>
* Merge branch 'python-discord:main' into patreonGravatar mathstrains212021-11-14-350/+816
|\
| * Change log level from `WARNING` to `DEBUG`. (#1950)Gravatar TizzySaurus2021-11-11-3/+1
| |
| * Merge PR #1947: Fix `!infractions by me`Gravatar TizzySaurus2021-11-10-1/+1
| | | | | | Put the literal converter before the Member converter so that "me"/"m" isn't attempted to be converted to a Member.
| * Merge pull request #1926 from python-discord/infractions-by-commandGravatar Johannes Christ2021-11-10-2/+58
| |\ | | | | | | Add support for `!infractions by <m|me|uid>`
| | * Merge branch 'main' into infractions-by-commandGravatar Boris Muratov2021-11-10-558/+1046
| | |\ | | |/ | |/|
| * | commands: add pip as an alias to pypi (#1942)Gravatar aru2021-11-10-1/+1
| | | | | | | | | Co-authored-by: Xithrius <[email protected]>
| * | Merge pull request #1946 from python-discord/refactor-modlogGravatar Matteo Bertucci2021-11-09-19/+21
| |\ \ | | | | | | | | Refactor ModLog
| | * | Listen to only on_raw_message_deleteGravatar Qwerty-1332021-11-09-19/+21
| | | |
| | * | Call the appropriate function in the raw listenerGravatar Qwerty-1332021-11-09-1/+1
| |/ /
| * | Merge pull request #1944 from python-discord/ignore-botGravatar ChrisJL2021-11-09-1/+1
| |\ \ | | | | | | | | Filter the same kind of messages in both message deletion listeners.
| | * | Filter the same messages in both listeners.Gravatar Qwerty-1332021-11-09-1/+1
| |/ /
| * | Only re-run filters in `on_message_update` if contents/attachments changed ↵Gravatar TizzySaurus2021-11-06-2/+10
| | | | | | | | | | | | (#1937)
| * | Merge pull request from GHSA-j8c3-8x46-8pp6Gravatar Hassan Abouelela2021-11-05-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Don't Exit Token Filtering Early On URLs The token filtering function would exit early if it detected a URL within the message, but it made no extra checks to ensure there weren't other tokens within that message that would trigger it. This made sense when the filtering logic was written, but it's been modified since to introduce this bug. Regression tests included. Signed-off-by: Hassan Abouelela <[email protected]> * Links Advisory In Token Filter Tests Adds a link to the advisory with reasoning for the existence of the test. Signed-off-by: Hassan Abouelela <[email protected]>
| * | Add sql-fstring tagGravatar Janine vN2021-11-04-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add sql-fstring tag * Correct link and wording * Correction to grammar and wording Also adds a semicolon * Add missing " Co-authored-by: Bluenix <[email protected]> Co-authored-by: TizzySaurus <[email protected]>
| * | Migrate to `og_blurple` (#1901)Gravatar TizzySaurus2021-11-01-38/+38
| | | | | | | | | Migrate to `og_blurple`
| * | Merge pull request #1910 from ↵Gravatar ChrisJL2021-10-31-2/+7
| |\ \ | | | | | | | | | | | | | | | | python-discord/consider-parent-channels-when-checking-mod-channels Consider parent channels when checking mod channels
| | * | consider parent channels when checking mod channelsGravatar Chris Lovering2021-10-31-2/+7
| |/ /
| * | Merge pull request #1923 from Lainika/GH-1873_Fix_embedsGravatar Karlis Suvi2021-10-31-2/+1
| |\ \
| | * | GH-1873 Fix BigBrother embedsGravatar Lainika2021-10-31-2/+1
| |/ / | | | | | | | | | Move text from footer to description.
| * | Merge pull request #1793 from python-discord/cleanreworkGravatar Johannes Christ2021-10-30-274/+614
| |\ \ | | | | | | | | Clean Cog Rework
| | * \ Merge branch 'main' into cleanreworkGravatar wookie1842021-10-30-10/+64
| | |\ \ | | |/ / | |/| |
| * | | Merge pull request #1909 from python-discord/pin-consistencyGravatar Johannes Christ2021-10-28-3/+9
| |\ \ \ | | | | | | | | | | Unpin All Messages When Moving Help Channels
| | * \ \ Merge branch 'main' into pin-consistencyGravatar Johannes Christ2021-10-28-7/+55
| | |\ \ \ | | |/ / / | |/| | |
| | * | | Unpin All Messages When Moving Help ChannelsGravatar Hassan Abouelela2021-10-24-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Occasional hiccups in the Discord API would cause unpinning in help channel to sometimes fails. This gets around that by unpinning all messages when making the channel available. Signed-off-by: Hassan Abouelela <[email protected]>
| | | * | Remove channel limitation with time rangeGravatar mbaruh2021-10-25-3/+0
| | | | | | | | | | | | | | | | | | | | Discussion in the pull request raised some legitimate use cases for supplying a time range for multiple channels (e.g clean the last couple of minutes instead of specifying number of messages to traverse).
| | | * | Update Age converter to use TZ aware datetimeGravatar mbaruh2021-10-25-1/+1
| | | | |
| | | * | Merge branch 'main' into cleanreworkGravatar mbaruh2021-10-25-221/+255
| | | |\ \ | | | |/ / | | |/| |
| | | * | Improve documentation of global variablesGravatar mbaruh2021-10-25-3/+5
| | | | |
| | | * | Apply requested style changesGravatar mbaruh2021-10-25-6/+4
| | | | |
| | | * | Adjust docstring to #1876Gravatar mbaruh2021-10-25-12/+11
| | | | |
| | | * | Merge branch 'main' into cleanreworkGravatar mbaruh2021-10-17-1144/+1955
| | | |\ \
| | | * | | Indentation, type-hint, and documentation fixesGravatar mbaruh2021-09-20-31/+32
| | | | | |
| | | * | | End clean on unexpected errorsGravatar mbaruh2021-09-11-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a cog_command_error method that sets cleaning to False when a command ends on an exception. I don't have anything in mind that might cause this, but it will ensure that in any case the cog will still be usable.
| | | * | | Handle Regex converter errorsGravatar mbaruh2021-09-11-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | Handle cases where there are no enclosing backticks, and where the regex pattern is invalid.
| | | * | | Fix incorrect cache usageGravatar mbaruh2021-09-08-1/+1
| | | | | |
| | | * | | Switch `users` and `traverse` in main commandGravatar mbaruh2021-09-08-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | When providing a user ID it would clash with `traverse` which came first.
| | | * | | Fix delete orderGravatar mbaruh2021-09-07-9/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case of old messages, it would delete the old messages first, and only then bulk delete the remainder, which affected logging. This commit corrects the deletion order.
| | | * | | Improve responsesGravatar mbaruh2021-09-07-22/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Tells the user if clean cancel was attempted with no ongoing clean. - Fixes MaxConcurrencyReached call bug. There was a missing argument, and it shouldn't invoke the help embed anyway, so it's now a message. - Some code refactoring.
| | | * | | Fix regex searchGravatar mbaruh2021-09-07-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The regex was lowercased, even though regex patterns are case sensitive. Also adds the DOTALL flag.
| | | * | | Code and comments polishGravatar mbaruh2021-09-07-25/+21
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Shivansh-007 <[email protected]>
| | | * | | Changed regex formatting to wrapped in backticksGravatar mbaruh2021-08-31-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After discussion, backticks seems like the preferrable formatting as it also cancels Discord's formatting. Additionally removed the Optionals from the last args in the commands, to not silently ignore incorrect input.
| | | * | | Disallow time range cleaning in multiple channelsGravatar mbaruh2021-08-31-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | Cleaning in the same time range across several channels seems like an arbitrary decision.
| | | * | | Added master commandGravatar mbaruh2021-08-29-74/+156
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The subcommands are kept simple and with few arguments, as they deal with most cases and their usage shouldn't be cumbersome. However we might to clean by criteria of several functionalities offered by the subcommands, for example delete a specific user's messages but only those that contain a certain pattern. For this reason the top-level command can now accept all arguments available in any of the subcommands, and will combine the criteria. Because the channels list has to be the last argument in order to accept either a list of channel or "*", I had to force a specific pattern in the regex argument for it to not consume the first channel specified. The regex argument must now have an "r" prefix and be enclosed in single quotes. For example: r'\d+'. For patterns with spaces the whole thing still needs to be enclosed in double quotes. For consistency the `clean regex` subcommand was changed to require the same.
| | | * | | Handle reacted message being deletedGravatar mbaruh2021-08-29-1/+3
| | | | | |
| | | * | | Use a cog-wide role checkGravatar mbaruh2021-08-29-8/+4
| | | | | |
| | | * | | Restrict until and between to a single channelGravatar mbaruh2021-08-29-12/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The subcommands should stay simple and answer the most common use cases. Deleting all messages within a time range across many channels seems esoteric and gives just more room for mistakes.
| | | * | | `until` and `between` overhaulGravatar mbaruh2021-08-29-53/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - The two subcommands can now accept a time delta and an ISO date time in addition to messages. - The two limits are now exclusive. Meaning cleaning until a message will not delete that message. - Added a separate predicate for the `until` case, as the combination of that command and cache usage would result in incorrect behavior. Additionally, deleting from cache now correctly traverses only `traverse` messages, rather than trying to delete `traverse` messages.
| | | * | | Send message when no messages foundGravatar mbaruh2021-08-28-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes the clean command to send a message instead of raising BadArgument when no messages are found. Not finding messages is not an error, and doesn't necessarily require the help embed to spring up, just that the parameters might need tweaking.
| | | * | | Add checkmark after command completes in mod channelsGravatar mbaruh2021-08-28-1/+4
| | | | | |