aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | | | | | | | | | | Reduce API calls in `!dmrelay`.Gravatar Den42002021-03-28-15/+13
| | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | Merge remote-tracking branch 'refs/remotes/origin/feat/dmrelay' into ↵Gravatar Den42002021-03-28-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | feat/dmrelay
| | * | | | | | | | | | | | | | | | | | | | | Improve `DMRelay` cog description.Gravatar Dennis Pham2021-03-28-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joe Banks <[email protected]>
| * | | | | | | | | | | | | | | | | | | | | | Force cache to update for user history.Gravatar Den42002021-03-28-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, the user would have to send a DM to the bot after startup for the bot to realize there is DM history with that specific user. Now, we force a cache refresh when a moderator invokes `!dmrelay`, so this shouldn't be an issue anymore.
| * | | | | | | | | | | | | | | | | | | | | | Account for requesting the bot's DMs with itself.Gravatar Den42002021-03-28-1/+1
| |/ / / / / / / / / / / / / / / / / / / / /
| * | | | | | | | | | | | | | | | | | | | | Added a newline to space out some code.Gravatar Dennis Pham2021-03-28-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joe Banks <[email protected]>
| * | | | | | | | | | | | | | | | | | | | | Restrict DMRelay cog to moderators only.Gravatar Den42002021-03-27-2/+6
| | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | Create the new DMRelay cog.Gravatar Den42002021-03-27-0/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Includes the `!dmrelay` command, allowing moderators to relay direct messages between the bot and other users.
| * | | | | | | | | | | | | | | | | | | | | Fix tests for paste uploads.Gravatar Den42002021-03-27-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Accounts for no redirects on extensions that are not `.py`.
| * | | | | | | | | | | | | | | | | | | | | Remove the old DMRelay cog.Gravatar Den42002021-03-27-132/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The moderation team has decided that this may come in conflict with Discord's ToS, and it does not serve too much of a purpose anymore. It was fun while it lasted!
| * | | | | | | | | | | | | | | | | | | | | Restrict redirects for paste uploads with an extension that is not `.py`.Gravatar Den42002021-03-27-1/+7
|/ / / / / / / / / / / / / / / / / / / / /
| | * | | | | | | | | | | | | | | | | | | Merge: changes from 'upstream/main' & conflict resolutionGravatar kwzrd2021-03-28-399/+854
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflict in the lockfile resolved by re-locking the merged Pipfile. Conflict in Branding constants resolved by keeping my local version. Change in the cog's target branch to 'main' from 'master' is currently irrelevant as we targets a development branch anyway.
* | | | | | | | | | | | | | | | | | | | | Merge pull request #1466 from vcokltfre/discord-tagsGravatar Joe Banks2021-03-26-0/+22
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | feat: add new discord.py tags
| * | | | | | | | | | | | | | | | | | | | Merge branch 'main' into discord-tagsGravatar Joe Banks2021-03-26-60/+512
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | | Merge pull request #1478 from python-discord/wookie184-codeownersGravatar Joe Banks2021-03-20-0/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added myself to CODEOWNERS
| * | | | | | | | | | | | | | | | | | | | | Added myself to CODEOWNERSGravatar wookie1842021-03-20-0/+1
|/ / / / / / / / / / / / / / / / / / / / /
* | | | | | | | | | | | | | | | | | | | | Merge pull request #1477 from python-discord/add-nomination-voting-channelGravatar Boris Muratov2021-03-20-2/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added nomination voting channel to config
| * | | | | | | | | | | | | | | | | | | | | Added nomination voting channel to configGravatar wookie1842021-03-20-2/+4
|/ / / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also changed talentpool review cog to post there instead of mod-announcements
* | | | | | | | | | | | | | | | | | | | | Merge pull request #1475 from python-discord/mbaruh/review_fixGravatar Joe Banks2021-03-19-5/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix review formatting when there's only one infraction type
| * | | | | | | | | | | | | | | | | | | | | Fix review formatting when there's only one infraction typeGravatar Boris Muratov2021-03-19-5/+8
|/ / / / / / / / / / / / / / / / / / / / /
* | | | | | | | | | | | | | | | | | | | | Merge pull request #1459 from python-discord/mbaruh/autoreviewGravatar Joe Banks2021-03-19-31/+467
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Talentpool Autoreview
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'mbaruh/autoreview' of https://github.com/python-discord/bot ↵Gravatar Boris Muratov2021-03-19-229/+174
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into mbaruh/autoreview
| | * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into mbaruh/autoreviewGravatar Joe Banks2021-03-19-229/+174
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1435 from python-discord/swfarnsworth/infraction_messageGravatar Matteo Bertucci2021-03-17-26/+32
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instructions to dispute an infraction vary by infraction type.
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into swfarnsworth/infraction_messageGravatar Matteo Bertucci2021-03-17-429/+358
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1467 from ↵Gravatar Kieran Siek2021-03-17-0/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/info/bot-xr/code-block-null-byte Code block: remove null bytes before parsing AST
| * | | | | | | | | | | | | | | | | | | | | | | Merge branch 'main' into bug/info/bot-xr/code-block-null-byteGravatar Kieran Siek2021-03-17-3/+4
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | | | | | Merge pull request #1468 from ↵Gravatar Matteo Bertucci2021-03-16-3/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/info/bot-xk/defcon-threshold-none Account for defcon threshold being None when displaying server info
| * | | | | | | | | | | | | | | | | | | | | | | | Info: account for defcon threshold being NoneGravatar MarkKoz2021-03-15-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes BOT-XK
| * | | | | | | | | | | | | | | | | | | | | | | | Defcon: fix naming conflict between threshold cmd and attributeGravatar MarkKoz2021-03-15-2/+2
|/ / / / / / / / / / / / / / / / / / / / / / / /
| * / / / / / / / / / / / / / / / / / / / / / / Code block: remove null bytes before parsing ASTGravatar MarkKoz2021-03-15-0/+3
|/ / / / / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `ast.parse` raises a ValueError complaining that source code strings cannot contain null bytes. It seems like they may accidentally get pasted into Discord by users sometimes.
| * | | | | | | | | | | | | | | | | | | | | | Merge branch 'master' into swfarnsworth/infraction_messageGravatar Steele Farnsworth2021-03-06-452/+723
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | | | Made multiline concatenated string conform to a certain style. Gravatar Steele Farnsworth2021-03-06-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | That style is not currently enforced by the linter. Co-authored-by: Matteo Bertucci <[email protected]>
| * | | | | | | | | | | | | | | | | | | | | | | Update the tests to reflect changes in expected behavior.Gravatar swfarnsworth2021-02-28-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DM sent to infracted users now instructs them to DM modmail if they want to discuss non-ban infractions, so the tests now check if that instruction is present. Note that there already exists a superfluous test for note infractions, for which no DM is sent by design.
| * | | | | | | | | | | | | | | | | | | | | | | Instructions to dispute an infraction vary by infraction type.Gravatar swfarnsworth2021-02-25-24/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the user was instructed to email the appeals email for infraction types that don't remove one from the server. They are now instructed to DM ModMail except for Ban-type infractions. Also removed the URL string literal from the hyperlink to that URL.
| | | * | | | | | | | | | | | | | | | | | | | | Review commands now use the user ID instead of nomination IDGravatar Boris Muratov2021-03-19-27/+19
| | | |/ / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The user ID is much more accessible, and is usually what is used to obtain the nomination ID.
| | | * | | | | | | | | | | | | | | | | | | | Don't reschedule reviews that are long overdueGravatar Boris Muratov2021-03-19-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If it's been over a day overdue for a review, don't reschedule it. This is done in order to not fire reviews for all nominations which are over 30 days old when the auto-reviewing feature is merged.
| | | * | | | | | | | | | | | | | | | | | | | Mark as reviewed when nominee is off serverGravatar Boris Muratov2021-03-19-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is necessary as otherwise the bot would try to review them every time it restarts
| | | * | | | | | | | | | | | | | | | | | | | Default message in review when no nomination reason givenGravatar Boris Muratov2021-03-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Replace mentions for ID's in watchlist listsGravatar Boris Muratov2021-03-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Uncached mentions render as 'invalid' users on mobile, and with the list now showing the user's name we can now just show the ID without many problems.
| | | * | | | | | | | | | | | | | | | | | | | Properly await coroutine in post_reviewGravatar Boris Muratov2021-03-19-1/+3
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Use ctx.send instead of ctx.channel.sendGravatar Boris Muratov2021-03-19-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: ToxicKidz <[email protected]>
| | | * | | | | | | | | | | | | | | | | | | | Add additional logging to _review.pyGravatar Boris Muratov2021-03-19-0/+8
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Apply requested style and grammar changesGravatar Boris Muratov2021-03-19-2/+2
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Apply requested grammar and style changes.Gravatar Boris Muratov2021-03-13-8/+13
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Use log instead of erroneous self.logGravatar Boris Muratov2021-03-13-1/+1
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Improve string building for long linesGravatar Boris Muratov2021-03-12-9/+15
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Reviewer no longer subclasses SchedulerGravatar Boris Muratov2021-03-12-5/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It didn't make much sense for the Reviewer to subclasses Scheduler. The Scheduler has methods that don't make sense to use on the Reviewer directly. There is now a Scheduler object as an attribute of the Reviewer. Interacting with it is done by adding __contains__, cancel, and cancel_all methods.
| | | * | | | | | | | | | | | | | | | | | | | The 'seen vote' emoji is now a random ducky.Gravatar Boris Muratov2021-03-11-4/+15
| | | | | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | | | | Limit new commands to mods+Gravatar Boris Muratov2021-03-11-0/+2
| | | | | | | | | | | | | | | | | | | | | | |