aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Fix lintingGravatar TizzySaurus2021-10-05-0/+1
| | | Reverted accidental line deletion
* Fix if statement errorGravatar TizzySaurus2021-10-05-10/+9
| | | | Adds `messages_as_list` for easy indexing of `self.messages.values()` Also updated the comment above if statement and tidied some code
* Make attachments check explicitGravatar TizzySaurus2021-10-05-1/+1
| | | Co-authored-by: Matteo Bertucci <[email protected]>
* Improvements to anti-spam log messageGravatar TizzySaurus2021-10-05-3/+9
| | | | | | | - Removed redundant f-string - Made it so that messages with attachments will *always* upload to the log site. - Made it so that truncates messages will also upload to the paste site, for full viewing. - Fixed a previously unnoticed bug where truncating didn't account for extra characters of the ellipses.
* Merge pull request #1853 from python-discord/catch-403-from-Typing-callsGravatar Matteo Bertucci2021-10-04-22/+55
|\ | | | | Monkey patch http.send_typing to catch 403s
| * Move all monkey patches to their own fileGravatar Chris Lovering2021-10-04-23/+22
| |
| * Monkey patch http.send_typing to catch 403sGravatar Chris Lovering2021-10-04-1/+35
|/ | | | Sometimes discord turns off typing events by throwing 403's, so we should catch those
* Merge pull request #1841 from ↵Gravatar Johannes Christ2021-10-04-9/+66
|\ | | | | | | | | python-discord/allow-helpers-to-edit-their-own-nomination-reason Allow helpers to edit their own nomination reason
| * Merge branch 'main' into allow-helpers-to-edit-their-own-nomination-reasonGravatar Boris Muratov2021-10-04-61/+102
| |\ | |/ |/|
* | Merge pull request #1851 from python-discord/fix-hasattr-checkGravatar wookie1842021-10-03-2/+2
|\ \ | | | | | | Handle channel category being None in antispam/malware checks
| * | Handle channel category being None in antispam/malware checksGravatar wookie1842021-10-03-2/+2
|/ / | | | | | | The code already handled the attribute not existing (e.g from a DM channel), but didn't handle TextChannels not in a category
* | Merge pull request #1837 from python-discord/add-get-or-fetch-utilGravatar wookie1842021-10-03-59/+100
|\ \ | | | | | | Add get or fetch util
| * \ Merge branch 'main' into add-get-or-fetch-utilGravatar wookie1842021-10-03-5/+16
| |\ \ | |/ / |/| |
| * | Add new poetry tasks for pytestGravatar Chris Lovering2021-09-29-1/+2
| | | | | | | | | | | | | | | | | | Updated the test task to now run with --ff which runs failed tests from the last run first Added retest, which runs pytest with --lf this only runs the failed tests from the last test run
| * | Rename channel helper to be consistent with other helpersGravatar Chris Lovering2021-09-29-6/+6
| | |
| * | Mock fetch_member in tests that user get_or_fetch nowGravatar Chris Lovering2021-09-29-6/+15
| | |
| * | Migrate all Guild.get_member calls to get_or_fetch_memberGravatar Chris Lovering2021-09-29-46/+53
| | | | | | | | | | | | This protects us against the guild cache not being fully populated with members.
| * | Add get_or_fetch_member utilGravatar Chris Lovering2021-09-29-0/+24
| | | | | | | | | | | | This is now needed, as we're a large server it's not guaranteed that the member cache will always be fully populated.
| | * Check cache for Members and UsersGravatar Chris Lovering2021-10-03-5/+5
| | | | | | | | | | | | By swapping the isinstance to check for int, the else block now catches the case where target is Member or User, this allows for editting the nomination reason of members that are off server.
| | * Block helpers from editing nomination reasons in channels other than ↵Gravatar Chris Lovering2021-10-03-3/+7
| | | | | | | | | | | | #nominations
| | * Force kwargs when calling helper utilGravatar ChrisJL2021-10-03-1/+2
| | | | | | | | | Co-authored-by: Bluenix <[email protected]>
| | * Allow helpers to edit their own nomination reasonGravatar Chris Lovering2021-10-03-8/+60
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | This change will allow helpers to run the edit reason command in the Talentpool cog. To ensure that non-mod helpers can only edit their own reasons the interface for the reason command has been changed slightly. If nominee_or_nomination_id is a member or user, then the command edits the currently active nomination for that person. If it's an int, then use this to look up that nomination ID to edit. If no nominator is specified, assume the invoker is editing their own nomination reason. Otherwise, edit the reason from that specific nominator. Raise a permission error if a non-mod staff member invokes this command on a specific nomination ID, or with an nominator other than themselves.
* | python-news escape fixes (#1822)Gravatar TizzySaurus2021-09-29-5/+16
|/ | | | | | | | | | | | | * Fix escapes in python-news posts No longer escapes markdown inside of codeblocks or pre-escaped markdown. Co-authored-by: Ryu18 <[email protected]> * Add escaping to title of py-news posts * Fix typo Co-authored-by: Ryu18 <[email protected]>
* Merge pull request #1848 from python-discord/add-more-supported-symbols-to-otnGravatar Matteo Bertucci2021-09-28-3/+4
|\ | | | | Add LT and GT to supported otn chars
| * Add LT and GT to supported otn charsGravatar Chris Lovering2021-09-27-3/+4
|/
* Merge pull request #1825 from Numerlor/scheduling-create_task-initGravatar Xithrius2021-09-26-55/+85
|\ | | | | Use scheduling create_task util to create tasks
| * Merge branch 'main' into scheduling-create_task-initGravatar Xithrius2021-09-26-15/+44
| |\ | |/ |/|
* | Merge pull request #1846 from ↵Gravatar Johannes Christ2021-09-25-1/+14
|\ \ | | | | | | | | | | | | python-discord/suppress-notfound-messages-on-delete Suppress NotFound for batch deletion.
| * | Suppress NotFound for batch deletion.Gravatar Johannes Christ2021-09-24-1/+14
|/ / | | | | | | | | | | | | | | | | | | | | | | | | I will copy my comment from the code, as it explains why we want this: In the rare case where we found messages matching the spam filter across multiple channels, it is possible that a single channel will only contain a single message to delete. If that should be the case, discord.py will use the "delete single message" endpoint instead of the bulk delete endpoint, and the single message deletion endpoint will complain if you give it that does not exist. As this means that we have no other message to delete in this channel (and message deletes work per-channel), we can just log an exception and carry on with business.
* | Merge pull request #1843 from python-discord/fix-paste-site-inconsistencyGravatar Matteo Bertucci2021-09-23-1/+1
|\ \ | | | | | | Update paste-site URL in `!paste`
| * | Use full paste linkGravatar Izan2021-09-23-1/+1
|/ /
* | Merge pull request #1840 from python-discord/mbaruh-patch-1Gravatar ChrisJL2021-09-21-1/+1
|\ \ | | | | | | Add `cog` as an alias to extensions command
| * | Add `cog` as an alias to extensions commandGravatar Boris Muratov2021-09-21-1/+1
|/ /
* | Merge pull request #1838 from python-discord/remove-coverallsGravatar Joe Banks2021-09-20-7/+0
|\ \
| * | Remove coveralls from lint-testGravatar Izan2021-09-20-7/+0
|/ /
* | Merge pull request #1820 from python-discord/add-metricity-to-composeGravatar ChrisJL2021-09-17-2/+23
|\ \ | | | | | | Add metricity to docker-compose
| * | Remove duplicate roles when defining allowed_mentionsGravatar Chris Lovering2021-09-17-1/+1
| | |
| * | Add metricity to docker-composeGravatar Chris Lovering2021-09-17-1/+22
|/ / | | | | | | | | | | By adding metricity to the compose, we allow it to migrate itself, rather than needing the site to do it. Defaulting 'USE_METRICITY' to false means that it will run migrations, but not actually start the bot. This means we don't add another service that needs to run all the time, which could impact some contribs on lower powered hardware.
* | Merge pull request #1826 from python-discord/ignore-votingGravatar ChrisJL2021-09-11-3/+5
|\ \ | | | | | | Adds Core Dev Voting To Changelog Blacklist
| * | Adds Core Dev Voting To Changelog BlacklistGravatar Hassan Abouelela2021-09-11-3/+5
|/ / | | | | | | | | | | | | Adds the channel used for voting on contributors to the message changelog blacklist. Signed-off-by: Hassan Abouelela <[email protected]>
| * Use create_task util instead of asyncio.create_taskGravatar Numerlor2021-09-11-8/+6
| | | | | | | | | | The util attaches an error logging callback instead of relying on python's exception logging which only occurs when the task is destroyed
| * Use scheduling create_task util instead of creating from loop directlyGravatar Numerlor2021-09-11-47/+79
|/ | | | | The util attaches an error logging callback instead of relying on python's exception logging which only occurs when the task is destroyed
* Merge pull request #1788 from ↵Gravatar ChrisJL2021-09-09-8/+4
|\ | | | | | | | | python-discord/Only-check-domain-filters-against-URL-like-parts-of-a-message Only check URL-like objects against domain filters
| * Merge branch 'main' into ↵Gravatar ChrisJL2021-09-09-240/+450
| |\ | |/ |/| | | Only-check-domain-filters-against-URL-like-parts-of-a-message
* | Merge pull request #1819 from python-discord/string-formatting-tagGravatar ChrisJL2021-09-09-0/+24
|\ \ | | | | | | Add string-formatting tag
| * \ Merge branch 'main' into string-formatting-tagGravatar ChrisJL2021-09-09-17/+12
| |\ \ | |/ / |/| |
* | | Merge pull request #1813 from python-discord/declare-talentpool-cache-on-startupGravatar ChrisJL2021-09-08-17/+12
|\ \ \ | | | | | | | | Declare TalentPool.cache on init of cog
| * \ \ Merge branch 'main' into declare-talentpool-cache-on-startupGravatar ChrisJL2021-09-08-20/+52
| |\ \ \ | |/ / / |/| | |
| * | | Use - rather than 0 for number of nominees when cache isn't readyGravatar Chris Lovering2021-09-08-1/+1
| | | |
| * | | Remove previous nominations outputGravatar Chris Lovering2021-09-08-11/+0
| | | | | | | | | | | | | | | | This raised questions from helpers when they saw that someone had many previous nominations. There is no reason why a helper needs to see this information.