aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | | | 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
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
* | | | | | | Handle autoban filtering in DMs (#1914)Gravatar Boris Muratov2021-10-26-1/+2
| | | | | | | | | | | | | | | | | | | | | An autoban trigger being sent in DMs caused the ban to fail, but for it to still be registered in the database. That is becuase the ban command uses the `ctx.guild.ban` method, but in DMs `ctx.guild` is None. This commit solves it by overriding the `context.guild` field.
* | | | | | | Merge pull request #1893 from python-discord/filters/autobanGravatar Boris Muratov2021-10-25-7/+54
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Filtering: add autoban on specific reasons
| * | | | | | | Filtering: remove dangling empty quoteGravatar Matteo Bertucci2021-10-25-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: ChrisJL <[email protected]>
| * | | | | | | Filtering: update auto-ban commentsGravatar Matteo Bertucci2021-10-25-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: ChrisJL <[email protected]>
| * | | | | | | Filtering: fix ban flowGravatar Matteo Bertucci2021-10-25-2/+3
| | | | | | | |
| * | | | | | | Fltering: clean up autoban codeGravatar Matteo Bertucci2021-10-25-4/+4
| | | | | | | |
| * | | | | | | Filter list: move warning to #mod-metaGravatar Matteo Bertucci2021-10-25-4/+4
| | | | | | | |
| * | | | | | | Merge remote-tracking branch 'origin/main' into filters/autobanGravatar Matteo Bertucci2021-10-25-303/+360
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Filter list: send warning when autoban trigger is addedGravatar Matteo Bertucci2021-10-25-2/+11
| | | | | | | |
| * | | | | | | Filtering: update auto ban messageGravatar Matteo Bertucci2021-10-25-3/+15
| | | | | | | |
| * | | | | | | Infrac: prioritize mod over bot feedback msgGravatar Matteo Bertucci2021-10-25-8/+8
| | | | | | | |
| * | | | | | | Filtering: make autoban tempGravatar Matteo Bertucci2021-10-19-8/+4
| | | | | | | |
| * | | | | | | Filtering: do not try to convert to a memberGravatar Matteo Bertucci2021-10-19-2/+2
| | | | | | | |
| * | | | | | | Filtering: update auto-ban messageGravatar Matteo Bertucci2021-10-19-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Boris Muratov <[email protected]>
| * | | | | | | Filtering: add autoban on specific reasonsGravatar Matteo Bertucci2021-10-19-0/+28
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to the increase in typo-squatting based phishing, we want to automatically ban users sending specific domain names. For that, this commit will automatically ban any user that trigger a filter which has `[autoban]` in its reason. That's it!
| | * | | | | 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
| | |\ \ \ \ | |_|/ / / / |/| | | | |
* | | | | | Merge pull request #1908 from python-discord/fix-tz-issueGravatar Hassan Abouelela2021-10-23-2/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use datetime.fromtimestamp so we pass relativedelta a datetime object…
| * | | | | | Use datetime.fromtimestamp so we pass relativedelta a datetime object, not ArrowGravatar Chris Lovering2021-10-22-2/+5
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Fixes #1907 Fixes BOT-1PA
* | | | | | Merge pull request #1906 from python-discord/fix-tzinfo-issueGravatar Hassan Abouelela2021-10-23-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use Arrow.fromtimestamp to get an aware datetime
| * | | | | | Use Arrow.fromtimestamp to get an aware datetimeGravatar Chris Lovering2021-10-22-2/+2
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #1905 Fixes BOT-1P9 datetime.fromtimestamp returned an naive datetime, so when comparing to the aware datetime from dateutil.parser.isoparse, it would raise an error.
* | | | | | Move to timezone aware datetimes (#1895)Gravatar Boris Muratov2021-10-22-113/+122
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move to timezone aware datetimes With the shift of the discord.py library to timezone aware datetimes, this commit changes datetimes throughout the bot to be in the UTC timezone accordingly. This has several advantages: - There's no need to discard the TZ every time the datetime of a Discord object is fetched. - Using TZ aware datetimes reduces the likelihood of silently adding bugs into the codebase (can't compare an aware datetime with a naive one). - Our DB already stores datetimes in UTC, but we've been discarding the TZ so far whenever we read from it. Specific places in the codebase continue using naive datetimes, mainly for UI purposes (for examples embed footers use naive datetimes to display local time). * Improve ISODateTime converter documentation Co-authored-by: Kieran Siek <[email protected]>
* | | | | | Merge pull request #1904 from python-discord/modlog/explicit-thread-nameGravatar ChrisJL2021-10-22-3/+12
|\ \ \ \ \ \ | | | | | | | | | | | | | | Modlog: explicitly write thread names
| * | | | | | Modlog: explicitly write thread namesGravatar Matteo Bertucci2021-10-22-3/+12
|/ / / / / / | | | | | | | | | | | | | | | | | | It seems like there could be some caching issue with threads causing to appear as deleted channels. Beside, we also want to keep the name of deleted threads around.
* | | | | | Merge pull request #1902 from python-discord/bump-licensesGravatar Matteo Bertucci2021-10-21-88/+95
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Bumps Pip Licences
| * | | | | Bumps Pip LicencesGravatar Hassan Abouelela2021-10-22-88/+95
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the pip-licences version to fix a breaking bug in the currently pinned version. Signed-off-by: Hassan Abouelela <[email protected]>
| | * | | 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.