aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | Improve cache iteration speedGravatar Boris Muratov2021-08-22-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | getitem based iteration included operations that aren't necessary when iterating over the cache continuously. Adding an iter method to the class seems to have improved iteration speed by several orders of magnitude.
| | * | | | | | Additional comments and tests for slicingGravatar mbaruh2021-08-21-2/+5
| | | | | | | |
| | * | | | | | Clean up codeGravatar mbaruh2021-08-21-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed unused import, corrected docstring, and removed unnedded type annotation.
| | * | | | | | Fix MessageCache slicing bugs, improve testsGravatar mbaruh2021-08-21-26/+41
| | | | | | | |
| | * | | | | | Identify DeletionContext by members, not channelsGravatar mbaruh2021-08-21-17/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the anti-spam now works cross-channels, it makes no sense to identify it by the channel in which it was invoked. The DeletionContext class was changed to accept a frozenset of members, and the message_deletion_queue dict uses the frozensets as keys. DeletionContext still accepts a channel on creation, because while it might get added more channels, there's only one channel in which the mute message will be sent. Using members as the key can run into the issue of one member becoming irrelevant to the filter while others still are, resulting in another log message being sent, but it's an unlikely edge case since the users should be muted almost immediately, and we're currently not using any multi-member filters in the first place.
| | * | | | | | AntiSpam deletes from all spammed channelsGravatar mbaruh2021-08-21-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The anti-spam cog was amended to handle cross-channel spam.
| | * | | | | | AntiSpam modified to work with cacheGravatar mbaruh2021-08-21-9/+406
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The anti-spam cog now uses a cache instead of reading channel history. The cache is for all channels in the guild, and does not remove deleted messages. That means that the anti-spam logic now works cross-channel and counts deleted messages. The size of the cache is determined via a new field in the config YAML file. The cache was implemented as a separate class, MessageCache, which uses circular buffer logic. This allows for constant time addition and removal form either side, and lookup. The cache does not support removal from the middle of the cache. The cache additionally stores a mapping from message ID's to the index of the message in the cache, to allow constant time lookup by message ID. The commit additionally adds accompanying tests, and renames `cache.py` to `caching.py` to better distinguish it from the new `message_cache.py` and convey that it's for general caching utilities.
| | * | | | | | Move max_interval to initGravatar Boris Muratov2021-08-17-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The on_message event calculated the max interval value every time for no reason. The value is constant throughout the bot's up time.
| | | * | | | | Remove restriction on snowflake commandGravatar D0rs4n2021-08-23-4/+0
| |_|/ / / / / |/| | | | | |
* | | | | | | Merge pull request #1771 from python-discord/TizzySaurus-patch-1Gravatar Matteo Bertucci2021-08-23-0/+2
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Escape markdown in user's name for `!user` command
| * | | | | | Escape markdown in user's name for `!user` commandGravatar TizzySaurus2021-08-23-0/+2
|/ / / / / /
* | | | / / Update reminders command to use `UserMentionOrID` instead of ↵Gravatar TizzySaurus2021-08-23-4/+5
| |_|_|/ / |/| | | | | | | | | | | | | | `discord.Member` to fix greediness issues. (#1768)
* | | | | Update time format in reminder message to DAY_TIME from RELATIVE (#1766)Gravatar TizzySaurus2021-08-23-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | * Update time format in reminder message to DAY_TIME from RELATIVE Co-authored-by: Xithrius <[email protected]>
* | | | | Merge pull request #1762 from python-discord/metabase-sharingGravatar Xithrius2021-08-22-51/+70
|\ \ \ \ \ | | | | | | | | | | | | Metabase sharing
| * \ \ \ \ Merge branch 'main' into metabase-sharingGravatar Xithrius2021-08-22-174/+90
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #1742 from python-discord/remove-redundant-convertersGravatar Boris Muratov2021-08-22-166/+78
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove redundant converters and renamed FetchedMember to MemberOrUser
| * \ \ \ \ \ Merge branch 'main' into remove-redundant-convertersGravatar Boris Muratov2021-08-22-340/+574
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
* | | | | | | Merge pull request #1743 from python-discord/dawnofmidnight/site-urlsGravatar Boris Muratov2021-08-22-7/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fix: update urls in site cog
| * \ \ \ \ \ \ Merge branch 'main' into dawnofmidnight/site-urlsGravatar Boris Muratov2021-08-22-333/+567
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #1764 from python-discord/TizzySaurus-patch-1Gravatar Matteo Bertucci2021-08-22-1/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix edge-case of `user.joined_at` being `None` in userinfo command.
| * | | | | | | | Fix edge-case of `user.joined_at` being `None` in userinfo command.Gravatar TizzySaurus2021-08-22-1/+5
|/ / / / / / / /
| * | | | | | | fix: update urls in site cogGravatar dawnofmidnight2021-08-11-7/+7
| | | | | | | |
| | * | | | | | Update outdated commentGravatar Izan2021-08-22-1/+1
| | | | | | | |
| | * | | | | | Revert `search_user` typehint back to `MemberOrUser`Gravatar Izan2021-08-22-2/+2
| | | | | | | |
| | * | | | | | Update `infraction_search_group` typehint to `UserMentionOrId`Gravatar Izan2021-08-22-3/+3
| | | | | | | |
| | * | | | | | Attempt to get user from DB when discord.Object passed to search_userGravatar Izan2021-08-22-2/+6
| | | | | | | |
| | * | | | | | Remove redundant getattr protectionGravatar Izan2021-08-22-2/+2
| | | | | | | |
| | * | | | | | Remove redundant fetching of user codeGravatar Izan2021-08-22-11/+4
| | | | | | | |
| | * | | | | | Add missing call to `escape_markdown`Gravatar Izan2021-08-22-2/+6
| | | | | | | |
| | * | | | | | Re-add ability to search infractions of deleted accountGravatar Izan2021-08-22-6/+9
| | | | | | | |
| | * | | | | | Remove redundant functionGravatar Izan2021-08-22-13/+0
| | | | | | | |
| | * | | | | | Fix isinstance checkGravatar Izan2021-08-22-1/+1
| | | | | | | |
| | * | | | | | Fix importGravatar Izan2021-08-22-1/+1
| | | | | | | |
| | * | | | | | Fix import orderGravatar Izan2021-08-22-1/+1
| | | | | | | |
| | * | | | | | Rename converter from FetchedMember to MemberOrUserGravatar Izan2021-08-22-57/+58
| | | | | | | |
| | * | | | | | Remove unused importsGravatar TizzySaurus2021-08-22-4/+3
| | | | | | | |
| | * | | | | | Update to new convertersGravatar TizzySaurus2021-08-22-29/+21
| | | | | | | |
| | * | | | | | Remove converters made redundant by dpy V1.6's UserConverter updateGravatar TizzySaurus2021-08-22-72/+1
| | | | | | | |
| | | * | | | | Add ability to publish metabase questionsGravatar Chris Lovering2021-08-22-0/+14
| | | | | | | |
| | | * | | | | Trigger typing, to avoid wrapping whole func in a context managerGravatar Chris Lovering2021-08-22-30/+30
| | | | | | | |
| | | * | | | | Add alias for metabase exportGravatar Chris Lovering2021-08-22-1/+1
| | | | | | | |
| | | * | | | | Change metabase config to base url, rather than api urlGravatar Chris Lovering2021-08-22-9/+7
| | | | | | | |
| | | * | | | | Move metabase error handling to a cog error handlerGravatar Chris Lovering2021-08-22-26/+33
| |_|/ / / / / |/| | | | | |
* | | | | | | Merge pull request #1761 from python-discord/log-info-reminder-repliesGravatar ChrisJL2021-08-22-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Lowering log level to info for missing reminder replies.
| * | | | | | | Error to info log level on missing reminder message.Gravatar Xithrius2021-08-22-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #1749 from D0rs4n/pr/replindersGravatar Matteo Bertucci2021-08-22-16/+23
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Update reminders to reply instead of using a jump url to the origin message
| * \ \ \ \ \ \ Merge branch 'main' into pr/replindersGravatar Matteo Bertucci2021-08-22-302/+500
| |\ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
* | | | | | | | Merge pull request #1725 from Ryu1845/escape_md_newsGravatar Xithrius2021-08-22-2/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | added escape markdown in PythonNews
| * \ \ \ \ \ \ \ Merge branch 'main' into escape_md_newsGravatar Xithrius2021-08-22-324/+573
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #1756 from ↵Gravatar Matteo Bertucci2021-08-19-10/+10
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/Formatting-issues-on-the-!server-command Formatting issues on the !server command