bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Age	Lines
...
* \| \| \| \| \| \| \| \| \|	Bug fixes	Shivansh-007	2021-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.	Shivansh-007	2021-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.	Shivansh-007	2021-10-11	-4/+8
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Use tasks to fetch incidents channel webhook.	Shivansh-007	2021-10-11	-9/+13
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	On msg edits, edit the msg link embed rather than deleting it	Shivansh-007	2021-10-11	-4/+9
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Make incidents channel webhook a cog level attribute	Shivansh-007	2021-10-11	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This would not fetch it everytime.
* \| \| \| \| \| \| \| \| \|	Use str() rather than f string for single variable.	Shivansh-007	2021-10-11	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Makes the intent much more clear.
* \| \| \| \| \| \| \| \| \|	Don't allow more than 10 embeds per report.	Shivansh-007	2021-10-11	-53/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If more than 10 embeds found, just get the first 10 and ignore the rest.
* \| \| \| \| \| \| \| \| \|	Revert changes done by black.	Shivansh-007	2021-10-11	-43/+13
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Updates type hints for `message_link_embeds_cache`.	Shivansh-007	2021-10-11	-1/+7
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Refactors code.	Shivansh-007	2021-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.	Shivansh-007	2021-10-11	-2/+2
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Apply grammar and style changes.	Shivansh-007	2021-10-11	-13/+20
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Run black code formatter.	Shivansh-007	2021-10-11	-21/+53
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Ignore N802 in 'asyncSetUp' and 'asyncTearDown' function in test_incidents.py	Shivansh-007	2021-10-11	-2/+2
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Allign comments to maintain readability	Shivansh-007	2021-10-11	-3/+3
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Modify tests to support redis cache, done with the help @SebastiaanZ	Shivansh-007	2021-10-11	-16/+16
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Run webhook message deletion if webhook_msg_id var is True	Shivansh-007	2021-10-11	-5/+7
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Update regex to support all message links i.e. support for 'app', 'canary', ↵	Shivansh-007	2021-10-11	-3/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	'ptb'
* \| \| \| \| \| \| \| \| \|	Add a docstring to 'send_webhooks' function	Shivansh-007	2021-10-11	-0/+9
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	If message content more than 500 characters shorten it done to 300 characters	Shivansh-007	2021-10-11	-24/+34
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Rollback to changes which aren't required	Shivansh-007	2021-10-11	-6/+6
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Fix tests according to the changes done to incidents.py	Shivansh-007	2021-10-11	-3/+19
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Use str() when checking for message.content	Shivansh-007	2021-10-11	-1/+1
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Use MessageConverter to find messages	Shivansh-007	2021-10-11	-25/+35
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Send multiple webhook messages in case of more than 10 message links	Shivansh-007	2021-10-11	-9/+22
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Add incidents webhook to default config template	Shivansh-007	2021-10-11	-0/+1
\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Send webhook embed containing information about the message if there is a ↵	Shivansh-007	2021-10-11	-4/+70
\|/ / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	message link in the incident report
* \| \| \| \| \| \| \| \|	Merge pull request #1865 from python-discord/fix-bot-1858	ChrisJL	2021-10-10	-1/+9
\|\ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|/ / / \|/\| \| \| \| \| \| \| \|	Handle 400 when setting pre-existing doc package
\| * \| \| \| \| \| \| \|	Fix typo in bot/exts/info/doc/_cog.py	Richard Si	2021-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 package	Richard Si	2021-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-subdomains	Joe Banks	2021-10-06	-3/+3
\|\ \ \ \ \ \ \ \
\| * \ \ \ \ \ \ \	Merge branch 'main' into kill-subdomains	Joe Banks	2021-10-06	-0/+4
\| \|\ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #1862 from python-discord/add-appeals-categories	ChrisJL	2021-10-06	-0/+4
\|\ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / \|/\| \| \| \| \| \| \| \|	Add appeal categories to mod categories
\| * \| \| \| \| \| \| \|	Add appeal categories to mod categories	Chris Lovering	2021-10-06	-0/+4
\|/ / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This allows us to run moderation commands in the appeal categories
\| * / / / / / /	Kill subdomains from configuration file.	Johannes Christ	2021-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-dm	Matteo Bertucci	2021-10-06	-10/+8
\|\ \ \ \ \ \ \ \| \|_\|_\|/ / / / \|/\| \| \| \| \| \|	Revert "Apply infractions before DMing"
\| * \| \| \| \| \|	Revert "Apply infractions before DMing"	Matteo Bertucci	2021-10-06	-10/+8
\|/ / / / / /
* \| \| \| \| \|	Merge pull request #1798 from python-discord/ignore-infra-mods-errors	Johannes Christ	2021-10-05	-7/+22
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Prevent role hierarchy issues with infractions
\| * \ \ \ \ \	Merge branch 'main' into ignore-infra-mods-errors	TizzySaurus	2021-10-05	-8/+10
\| \|\ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
* \| \| \| \| \| \|	Merge pull request #1806 from python-discord/infract-then-dm	Johannes Christ	2021-10-05	-8/+10
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Apply infractions before DMing
\| * \ \ \ \ \ \	Merge branch 'main' into infract-then-dm	Joe Banks	2021-10-05	-190/+420
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Apply infractions before DMing	Izan	2021-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-errors	Joe Banks	2021-10-05	-323/+653
\| \| \|\ \ \ \ \ \ \| \|_\|/ / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #1854 from python-discord/minor-changes-to-typing-patch	Johannes Christ	2021-10-05	-3/+3
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use utcnow() and lower a logging level in patch_typing
\| * \ \ \ \ \ \ \	Merge branch 'main' into minor-changes-to-typing-patch	Johannes Christ	2021-10-05	-20/+48
\| \|\ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #1855 from python-discord/antispam-log-improvements	Matteo Bertucci	2021-10-05	-8/+14
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Improve anti-spam logs
\| * \ \ \ \ \ \ \ \	Merge branch 'main' into antispam-log-improvements	Matteo Bertucci	2021-10-05	-12/+34
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \|	Merge pull request #1824 from Numerlor/doc-fetch_inventory-error-badarg	wookie184	2021-10-05	-12/+34
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Gracefully handle invalid inventory content when adding a new doc package
\| * \ \ \ \ \ \ \ \ \	Merge branch 'main' into doc-fetch_inventory-error-badarg	wookie184	2021-10-05	-170/+372
\| \|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|