aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | 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.
| * | | | | | Wait until login before trying to use the bot api clientGravatar Chris Lovering2021-09-08-1/+5
| | | | | | |
| * | | | | | Declare and refresh TalentPool.cache on init of cogGravatar Chris Lovering2021-09-06-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids issues in the server cog trying to access it before it's assigned and refreshed. I also migrated to the tasks to `scheduling.create_task()` as the created tasks currently don't have any error handling they can hide errors in development until the task object is destroyed (if that occurs at all) which logs the exception. The scheduling alternative attaches a callback which logs exceptions to prevent this.
| | * | | | | Change formatting of examplesGravatar Janine vN2021-09-09-16/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After some discussion back and forth, I've adjusted how to display the examples and code bock to be of a more REPL-style. Additionally, a filler character for the "Center Me!" string is added to illustrate how exactly in centers it. This commit also adds some small styling changes.
| | * | | | | Add required newline to end of fileGravatar Janine vN2021-09-09-1/+1
| | | | | | |
| | * | | | | Add string-formatting tagGravatar Janine vN2021-09-09-0/+27
| |/ / / / / |/| | | | | | | | | | | | | | | | | Adds a tag to show the string formatting mini language
* | | | | | Add support for `!infraction last` (#1804)Gravatar TizzySaurus2021-09-08-19/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add support for `!infraction last` `!infraction last` will view details of the last infraction the user made. Also changes the `Infraction` converter to use the `expanded` endpoint 1) Added `InvalidInfraction` to `bot.errors` - called when value passed to `Infraction` converter is invalid 2) Improved error messages for when an invalid infraction is passed to `!infraction {infr_id}` 3) Improved some other messages/docstrings etc. Co-authored-by: Xithrius <[email protected]>
* | | | | | Merge pull request #1814 from python-discord/Expand-fitlers-to-.gg-Gravatar Matteo Bertucci2021-09-06-1/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Add .gg/ to invite filter regex
| * | | | | Add .gg/ to invite filter regexGravatar Chris Lovering2021-09-06-1/+2
|/ / / / /
| * | | | Refactor & simplifiy domain filter checkGravatar Chris Lovering2021-09-06-8/+3
| | | | |
| * | | | Only check URL-like objects against domain filtersGravatar Chris Lovering2021-08-28-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously a message such as 'https://google.com hello! flask.request.method' would be filtered due to us filtering the url shortener t.me. This commit changes to logic so that we only check parts of the messages that matched the URL regex against our blacklist, to avoid these false-positives.
| | * | | Use `isinstance` instead of `hasattr` to determine if `Member`Gravatar Izan2021-10-05-2/+2
| | | | |
| | * | | Fix testsGravatar Izan2021-09-01-6/+9
| | | | |
| | * | | Fix role hierarchy checkGravatar Izan2021-09-01-6/+6
| | | | | | | | | | | | | | | | | | | | Now uses `>=` instead of `>`, as is meant to happen.
| | * | | Prevent infractions raising error due to role hierarchyGravatar Izan2021-09-01-0/+12
| | | | | | | | | | | | | | | | | | | | Now explicitly states that the bot is unable to starify/kick/ban someone who's higher in the role hierarchy
| | | * | Test incidents: correct avatar referenceGravatar Matteo Bertucci2021-10-05-1/+1
| | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-27 21:49:33 +0000