aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | * | | Refactor code, correct loggingGravatar mbaruh2021-08-28-96/+133
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit further splits the bulky _clean_messages function, and all its helper functions are grouped together in the same region. Additionally, this commit fixes logging by logging only the messages that have been successfully deleted, before being possibly interrupted by the cancel command.
| | | * | | Rename "amount" argument to "traverse"Gravatar mbaruh2021-08-28-24/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This name as been confusing moderators for a while now. "amount" sounds like this is the amount of messages that the bot will try to delete, and keep going until it reaches that number. In reality it's the amount of latest messages per channel the bot will traverse. Hopefully the new name conveys that better.
| | | * | | Change cache usageGravatar mbaruh2021-08-28-18/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The cache is used only when all channels are used, as before. Unlike before, using all channels requires using "*" in the channels argument. Before all channels would be used if use_cache was set to True. Using all channels uses the cache by default. To traverse every single text channel in the server, setting use_cache to False is required in the command.
| | | * | | Don't delete clean cancel embed in mod channelGravatar mbaruh2021-08-27-1/+4
| | | | | |
| | | * | | Change `from-to` primary name to `between`Gravatar mbaruh2021-08-27-2/+2
| | | | | |
| | | * | | Correct logging commentGravatar mbaruh2021-08-27-1/+1
| | | | | |
| | | * | | Move setting cleaning flag to correct lineGravatar mbaruh2021-08-27-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Between the concurrency check and setting the cleaning flag to True there was an await statement, which could potentially cause race conditions.The setting of the flag was moved to right below the concurrency check.
| | | * | | Move clean logging to a helper functionGravatar mbaruh2021-08-27-11/+11
| | | | | |
| | | * | | Moved clean cog to moderation extGravatar mbaruh2021-08-27-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | The cog is moderation related and all commands are exclusive to moderators.
| | | * | | Merge branch 'main' into cleanreworkGravatar mbaruh2021-08-27-933/+1826
| | | |\ \ \
| | | * | | | Fix references to kwarg after renaming in clean commandGravatar Chris Lovering2021-07-28-2/+2
| | | | | | |
| | | * | | | Rely on error handler for sending input errors to userGravatar Chris Lovering2021-07-28-45/+9
| | | | | | |
| | | * | | | simplify range predicate for clean commandGravatar Chris Lovering2021-07-28-1/+1
| | | | | | |
| | | * | | | Return empty containers if clean is cancelledGravatar Chris Lovering2021-07-28-2/+2
| | | | | | |
| | | * | | | Make is_older_than_14d a static methodGravatar Chris Lovering2021-07-26-11/+12
| | | | | | |
| | | * | | | Update _get_messages_from_channels return typeGravatar Chris Lovering2021-07-26-2/+2
| | | | | | |
| | | * | | | Merge branch 'main' into cleanreworkGravatar ChrisJL2021-07-26-2344/+5485
| | | |\ \ \ \
| | | * | | | | document snowflake check betterGravatar Senjan212021-04-16-4/+12
| | | | | | | |
| | | * | | | | Don't delete invocation in mod channelGravatar Senjan212021-04-16-7/+10
| | | | | | | |
| | | * | | | | Better response wording, added aliasGravatar Senjan212021-04-16-2/+2
| | | | | | | |
| | | * | | | | make predicate_range inclusiveGravatar Senjan212021-04-16-1/+1
| | | | | | | |
| | | * | | | | Naming changes for better self documentationGravatar Senjan212021-04-16-5/+5
| | | | | | | |
| | | * | | | | simplify use_cache varGravatar Senjan212021-04-16-6/+2
| | | | | | | |
| | | * | | | | Use correct kwarg for channel.historyGravatar Senjan212021-04-16-1/+1
| | | | | | | |
| | | * | | | | replace lambda with list in defaultdictGravatar Senjan212021-04-16-2/+2
| | | | | | | |
| | | * | | | | Merge branch 'main' into cleanreworkGravatar Senjan212021-04-15-2379/+4200
| | | |\ \ \ \ \
| | | * | | | | | swap predicate save order for correct deletionGravatar Senjan212021-04-15-4/+4
| | | | | | | | |
| | | * | | | | | Change typing, remove `range` aliasGravatar Senjan212021-03-04-3/+3
| | | | | | | | |
| | | * | | | | | Merge remote-tracking branch 'origin/master' into cleanreworkGravatar Senjan212021-03-04-170/+249
| | | |\ \ \ \ \ \
| | | * \ \ \ \ \ \ Merge remote-tracking branch 'origin/master' into cleanreworkGravatar Senjan212021-03-02-2961/+4031
| | | |\ \ \ \ \ \ \
| | | * | | | | | | | set `self.cleaning` to False once done cleaningGravatar Senjan212021-02-24-0/+2
| | | | | | | | | | |
| | | * | | | | | | | Implement range clean commandGravatar Senjan212021-02-23-6/+57
| | | | | | | | | | |
| | | * | | | | | | | Introduce cache to cleaning as well as fix cancelGravatar Senjan212021-02-23-36/+89
| | | | | | | | | | |
| | | * | | | | | | | rename command `messages` to `until`Gravatar Senjan212020-11-22-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | new name should be more selfexplanatory
| | | * | | | | | | | rework clean to fully use `delete_messages` instead of `purge`Gravatar Senjan212020-11-22-26/+60
| | | | | | | | | | |
| | | | | | | | | | * Unify infraction embed titleGravatar Izan2021-11-10-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Embed for `!infractions by` no longer has the author in codeblock.
| | | | | | | | | | * Add missing newline after region commentGravatar Izan2021-11-08-0/+1
| | | | | | | | | | |
| | | | | | | | | | * Improve ordering logic in API requestGravatar Izan2021-11-03-1/+6
| | | | | | | | | | |
| | | | | | | | | | * Address ReviewGravatar Izan2021-11-02-16/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add `format_infraction_count` and migrate - Improve logic for `actor` being `"m"`/`"me"` - Rename `search_by_user` to `search_by_actor` - Better Ordering Logic (thanks @ChrisLovering) - Make embed title consistent with other search embeds
| | | | | | | | | | * Add support for `!infractions by <m|me|uid>`Gravatar Izan2021-11-01-0/+38
| | | | | | | | | | |
* | | | | | | | | | | Save another API callGravatar mathstrains212021-10-28-1/+1
| | | | | | | | | | |
* | | | | | | | | | | Change command function docstringsGravatar mathstrains212021-10-28-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (Help command uses docstrings, so make them user-friendly)
* | | | | | | | | | | Update patreon descriptionGravatar mathstrains212021-10-28-2/+2
| | | | | | | | | | |
* | | | | | | | | | | Save API callGravatar mathstrains212021-10-28-2/+2
| | | | | | | | | | |
* | | | | | | | | | | Fix capitalisationGravatar mathstrains212021-10-27-1/+1
| | | | | | | | | | |
* | | | | | | | | | | Unify linksGravatar mathstrains212021-10-27-2/+2
| | | | | | | | | | |
* | | | | | | | | | | embeds= -> embed=Gravatar mathstrains212021-10-27-2/+2
| | | | | | | | | | |
* | | | | | | | | | | `commands.context` -> `commands.Context`Gravatar mathstrains212021-10-27-2/+2
| | | | | | | | | | |
* | | | | | | | | | | Adjust docstringsGravatar mathstrains212021-10-27-2/+2
| | | | | | | | | | |
* | | | | | | | | | | Rename some variablesGravatar mathstrains212021-10-27-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Rename some variables" This reverts commit 2f5aeb06e816240b08453b5cf327c6b77958a3ab. Fix variable names (again) And again