aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Handle discord.errors.NotFound while deleting msg link webhook embedsGravatar Shivansh-0072021-10-11-1/+4
|
* Remove leading whitespace from msg link embed contentGravatar Shivansh-0072021-10-11-1/+1
|
* Rename 'send_webhooks' to 'send_message_link_embed'Gravatar Shivansh-0072021-10-11-3/+3
| | | | Co-authored-by: Boris Muratov <[email protected]>
* Appy requested grammar changes.Gravatar Shivansh-0072021-10-11-5/+5
| | | | Co-authored-by: Boris Muratov <[email protected]>
* Remove redundant codeGravatar Shivansh-0072021-10-11-4/+2
|
* Rework message link embed.Gravatar Shivansh-0072021-10-11-7/+9
| | | | | | - Instead of default black colour, use gold to give it some shine! - Mention the channel also in the channel field. - Add message ID in footer, so it is easy to figure out for which message link is that embed.
* Bug fixesGravatar Shivansh-0072021-10-11-16/+14
| | | | | - `await` message link embeds cache get - don't double send webhook embeds (edit, send) on message edits
* Don't send errors, instead log them.Gravatar Shivansh-0072021-10-11-9/+9
| | | | Errors shouldn't be sent in #incidents. Instead, log them with log.exception and make the function return.
* Do required flake8 changes in docstrings.Gravatar Shivansh-0072021-10-11-4/+8
|
* Use tasks to fetch incidents channel webhook.Gravatar Shivansh-0072021-10-11-9/+13
|
* On msg edits, edit the msg link embed rather than deleting itGravatar Shivansh-0072021-10-11-4/+9
|
* Make incidents channel webhook a cog level attributeGravatar Shivansh-0072021-10-11-6/+6
| | | | This would not fetch it everytime.
* Use str() rather than f string for single variable.Gravatar Shivansh-0072021-10-11-1/+1
| | | | Makes the intent much more clear.
* Don't allow more than 10 embeds per report.Gravatar Shivansh-0072021-10-11-53/+32
| | | | If more than 10 embeds found, just get the first 10 and ignore the rest.
* Revert changes done by black.Gravatar Shivansh-0072021-10-11-43/+13
|
* Updates type hints for `message_link_embeds_cache`.Gravatar Shivansh-0072021-10-11-1/+7
|
* Refactors code.Gravatar Shivansh-0072021-10-11-26/+94
| | | | | | | | Earlier on message edit the message wasn't run through extract message links to see if new message links are added or if some got deleted. Similarly the cache was updated when a message got deleted. Now it makes extract message links a helper function and runs it on message edits and deletes in case there are some changes in the message links. This commit also updates the doc strings for functions according to the new changes done.
* Use `DiscordException` instead of broad exception clause.Gravatar Shivansh-0072021-10-11-2/+2
|
* Apply grammar and style changes.Gravatar Shivansh-0072021-10-11-13/+20
|
* Run black code formatter.Gravatar Shivansh-0072021-10-11-21/+53
|
* Ignore N802 in 'asyncSetUp' and 'asyncTearDown' function in test_incidents.pyGravatar Shivansh-0072021-10-11-2/+2
|
* Allign comments to maintain readabilityGravatar Shivansh-0072021-10-11-3/+3
|
* Modify tests to support redis cache, done with the help @SebastiaanZGravatar Shivansh-0072021-10-11-16/+16
|
* Run webhook message deletion if webhook_msg_id var is TrueGravatar Shivansh-0072021-10-11-5/+7
|
* Update regex to support all message links i.e. support for 'app', 'canary', ↵Gravatar Shivansh-0072021-10-11-3/+4
| | | | 'ptb'
* Add a docstring to 'send_webhooks' functionGravatar Shivansh-0072021-10-11-0/+9
|
* If message content more than 500 characters shorten it done to 300 charactersGravatar Shivansh-0072021-10-11-24/+34
|
* Rollback to changes which aren't requiredGravatar Shivansh-0072021-10-11-6/+6
|
* Fix tests according to the changes done to incidents.pyGravatar Shivansh-0072021-10-11-3/+19
|
* Use str() when checking for message.contentGravatar Shivansh-0072021-10-11-1/+1
|
* Use MessageConverter to find messagesGravatar Shivansh-0072021-10-11-25/+35
|
* Send multiple webhook messages in case of more than 10 message linksGravatar Shivansh-0072021-10-11-9/+22
|
* Add incidents webhook to default config templateGravatar Shivansh-0072021-10-11-0/+1
|
* Send webhook embed containing information about the message if there is a ↵Gravatar Shivansh-0072021-10-11-4/+70
| | | | message link in the incident report
* Merge pull request #1865 from python-discord/fix-bot-1858Gravatar ChrisJL2021-10-10-1/+9
|\ | | | | Handle 400 when setting pre-existing doc package
| * Fix typo in bot/exts/info/doc/_cog.py Gravatar Richard Si2021-10-10-1/+1
| | | | | | | | | | it only took 5 people looking through the diff ... Co-authored-by: Bluenix <[email protected]>
| * Handle 400 when setting pre-existing doc packageGravatar Richard Si2021-10-10-1/+9
|/ | | | | | | | | | If you run, for example: !doc setdoc black https://black.readthedocs/en/stable/objects.inv twice over. You'll get an unhelpful "According to the API, your request is malformed." error message back. This commit adds an error handler to catch the HTTP 400 and tell the user the package already exists.
* Merge pull request #1630 from python-discord/kill-subdomainsGravatar Joe Banks2021-10-06-3/+3
|\
| * Merge branch 'main' into kill-subdomainsGravatar Joe Banks2021-10-06-0/+4
| |\ | |/ |/|
* | Merge pull request #1862 from python-discord/add-appeals-categoriesGravatar ChrisJL2021-10-06-0/+4
|\ \ | | | | | | Add appeal categories to mod categories
| * | Add appeal categories to mod categoriesGravatar Chris Lovering2021-10-06-0/+4
|/ / | | | | | | This allows us to run moderation commands in the appeal categories
| * Kill subdomains from configuration file.Gravatar Johannes Christ2021-10-06-3/+3
|/ | | | | This will need a coordinated rollout with @jb3, in order to make the appropriate changes in Kubernetes beforehand.
* Merge pull request #1859 from python-discord/revert-1806-infract-then-dmGravatar Matteo Bertucci2021-10-06-10/+8
|\ | | | | Revert "Apply infractions before DMing"
| * Revert "Apply infractions before DMing"Gravatar Matteo Bertucci2021-10-06-10/+8
|/
* Merge pull request #1798 from python-discord/ignore-infra-mods-errorsGravatar Johannes Christ2021-10-05-7/+22
|\ | | | | Prevent role hierarchy issues with infractions
| * Merge branch 'main' into ignore-infra-mods-errorsGravatar TizzySaurus2021-10-05-8/+10
| |\ | |/ |/|
* | Merge pull request #1806 from python-discord/infract-then-dmGravatar Johannes Christ2021-10-05-8/+10
|\ \ | | | | | | Apply infractions before DMing
| * \ Merge branch 'main' into infract-then-dmGravatar Joe Banks2021-10-05-190/+420
| |\ \ | |/ / |/| |
| * | Apply infractions before DMingGravatar Izan2021-09-11-8/+10
| | | | | | | | | | | | Bot now attempts to infract before DMing, with the exception of kicks/bans, in order to prevent the user getting sent a message stating they were infracted when the infraction failed (and so they aren't).
| | * Merge branch 'main' into ignore-infra-mods-errorsGravatar Joe Banks2021-10-05-323/+653
| | |\ | |_|/ |/| |