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

	Commit message (Collapse)	Author	Age	Lines
...
* \| \| \|	Merge pull request #1569 from ↵	Hassan Abouelela	2021-05-06	-4/+8
\|\ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	python-discord/sebastiaan/prevent-accidental-addition-talentpool-cache Prevent accidental addition of users to talentpool cache when using `!talentpool get_review`
\| * \| \| \|	Prevent accidental addition of users to talentpool	Sebastiaan Zeeff	2021-05-06	-4/+8
\|/ / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Bug: When asking for a review for a user that isn't currently nominated using the `!talentpool get_review <userid>` command, the user is added to the talentpool `watched_users` cache, causing their messages to be relayed to the talentpool watch channel. Steps to reproduce: Use `!talentpool get_review <user_id>`, where `<user_id>` is the ID of a user not currently nominated. The command will correctly reply that the user isn't nominated, but their ID will be added as a key to the defaultdict nonetheless. Solution: replace all regular getitem usages with `.get(<user_id>)`, as the Reviewer should never insert IDs using the regular defaultdict path. Additional note: I've replaced all occurrences of regular getitem access into the defaultdict, even those that are normally not reachable with the id of a user that's currently not nominated, to prevent a future refactor from accidentally introducing this bug again.
* \| \| \|	Merge pull request #1561 from dawnofmidnight/patch-1	Matteo Bertucci	2021-05-05	-0/+7
\|\ \ \ \ \| \| \| \| \| \| \| \| \| \|	Add New Celebrities to `!superstarify`.
\| * \ \ \	Merge branch 'main' into patch-1	ChrisJL	2021-05-05	-6/+32
\| \|\ \ \ \ \| \|/ / / / \|/\| \| \| \|
* \| \| \| \|	Merge pull request #1531 from ToxicKidz/eval-redirect-output	ChrisJL	2021-05-05	-6/+32
\|\ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \|	Redirect output to bot-commands channel for the !eval command
\| * \ \ \ \	Merge branch 'main' into eval-redirect-output	ChrisJL	2021-05-05	-54/+1183
\| \|\ \ \ \ \ \| \|/ / / / / \|/\| \| \| \| \|
\| * \| \| \| \|	Merge branch 'eval-redirect-output' of https://github.com/ToxicKidz/bot into ↵	ToxicKidz	2021-05-05	-0/+1
\| \|\ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	eval-redirect-output
\| \| * \| \| \| \|	fix: Add a missing return statement	ToxicKidz	2021-04-22	-0/+1
\| \| \| \| \| \| \|
\| * \| \| \| \| \|	chore: Don't send a message when redirecting eval output which would ping ↵	ToxicKidz	2021-05-05	-3/+9
\| \|/ / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	users twice
\| * \| \| \| \|	chore: Redirect output to bot-commands channel for the eval command	ToxicKidz	2021-04-17	-5/+24
\| \| \| \| \| \|
\| \| * \| \| \|	Merge branch 'main' into patch-1	Matteo Bertucci	2021-05-05	-3/+27
\| \| \|\ \ \ \ \| \|_\|/ / / / \|/\| \| \| \| \|
* \| \| \| \| \|	Merge pull request #1441 from asleep-cult/master	ChrisJL	2021-05-05	-3/+3
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Unescape html escape characters in reddit text and titles
\| * \ \ \ \ \	Merge branch 'main' into master	ChrisJL	2021-05-05	-2596/+5693
\| \|\ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
* \| \| \| \| \| \|	Merge pull request #1517 from python-discord/is-vs-==	Matteo Bertucci	2021-05-05	-0/+24
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Create identity.md
\| * \ \ \ \ \ \	Merge branch 'main' into is-vs-==	Matteo Bertucci	2021-05-05	-881/+2915
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	fix: make requested changes	vcokltfre	2021-05-05	-10/+10
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	fix: remove the newline	vcokltfre	2021-05-05	-1/+0
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Create identity.md	SavagePastaMan	2021-04-12	-0/+25
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Tag to demonstrate the difference between `is` and `==`.
\| \| * \| \| \| \| \|	Resolved issues	asleep-cult	2021-04-15	-4/+2
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Import unescape directly	asleep-cult	2021-03-01	-3/+3
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Fix import order	asleep-cult	2021-03-01	-1/+1
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Fix pre-commit issues	asleep-cult	2021-03-01	-1/+1
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Make flake8 happy	asleep-cult	2021-03-01	-1/+1
\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \|	Unescape html escape characters in reddit text and titles	asleep-cult	2021-03-01	-2/+4
\| \| \| \| \| \| \| \|
\| \| \| \| * \| \| \|	Add more celebrities to stars.json	DawnOfMidnight	2021-05-04	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Celebrities: - The Weeknd - Ringo Starr - John Lennon - Guido Van Rossum - George Harrison - Darude
\| \| \| \| * \| \| \|	Update stars.json	DawnOfMidnight	2021-05-04	-0/+1
\| \|_\|_\|/ / / / \|/\| \| \| \| \| \|
\| \| \| \| * \| \|	Merge branch 'main' into laundmo-compose-restart-log	laundmo	2021-05-04	-2679/+5828
\| \| \| \| \|\ \ \ \| \|_\|_\|_\|/ / / \|/\| \| \| \| \| \|
\| \| \| \| * \| \|	Update docker-compose.yml	laundmo	2021-02-23	-0/+20
\| \| \| \| \| \| \|
\| \| \| \| \| * \|	Missed out removing one argument from pban	Shivansh-007	2021-05-07	-1/+1
\| \| \| \| \| \| \|
\| \| \| \| \| * \|	Apply request grammar changes.	Shivansh-007	2021-05-07	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Boris Muratov <[email protected]>
\| \| \| \| \| * \|	(infractions): Modify voice ban tests according to new changes in 3272605	Shivansh	2021-05-07	-2/+2
\| \| \| \| \| \| \|
\| \| \| \| \| * \|	(infractions): Apply temporary voice ban if duration specified by while ↵	Shivansh	2021-05-07	-3/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	voice banning user
\| \| \| \| \| * \|	(infractions): Remove purge days & add duration argument for pban	Shivansh	2021-05-07	-5/+4
\| \| \| \| \| \| \|
\| \| \| \| \| * \|	(infractions): Apply tempban if duration is specified while banning a user	Shivansh	2021-05-05	-3/+14
\| \|_\|_\|_\|/ / \|/\| \| \| \| \|
* \| \| \| \| \|	Merge pull request #1520 from python-discord/str-join-tag	Boris Muratov	2021-05-04	-0/+28
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Create str-join.md
\| * \ \ \ \ \	Merge branch 'main' into str-join-tag	Boris Muratov	2021-05-04	-881/+2887
\| \|\ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
* \| \| \| \| \| \|	Merge pull request #1556 from ToxicKidz/mod-ping-off-embed-timestamp	Matteo Bertucci	2021-05-01	-3/+5
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	feat: Use embed timestamp in modpings off
\| * \ \ \ \ \ \	Merge branch 'main' into mod-ping-off-embed-timestamp	ToxicKidz	2021-05-01	-19/+37
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #1555 from python-discord/bug/info/1553/snippet-404	Xithrius	2021-04-30	-14/+20
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Code Snippets: Fix Error Handling for HTTP Request Errors
\| * \ \ \ \ \ \ \	Merge branch 'main' into bug/info/1553/snippet-404	Xithrius	2021-04-30	-5/+17
\| \|\ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #1551 from dolphingarlic/master	ChrisJL	2021-04-30	-5/+17
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed the code snippets limits
\| * \ \ \ \ \ \ \ \	Merge branch 'main' into master	ChrisJL	2021-04-30	-19/+603
\| \|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Wait for cache to fill before redirecting	Andi Qu	2021-04-27	-0/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Redirect to #bot-commands if the message's length is > 1000	Andi Qu	2021-04-27	-5/+16
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Fixed the line limit and halved the char limit	Andi Qu	2021-04-27	-1/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Merge pull request #3 from python-discord/main	Andi Qu	2021-04-27	-0/+0
\| \|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #1028 from dolphingarlic/master
\| \| \| * \| \| \| \| \| \| \|	CodeSnippets: fix type annotations	MarkKoz	2021-04-28	-4/+5
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	CodeSnippets: add more detail to the request error message	MarkKoz	2021-04-28	-2/+3
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \|	CodeSnippets: use a lower log level for 404 responses	MarkKoz	2021-04-28	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Just cause a URL looks valid doesn't mean it will be valid, so a 404 is a normal and harmless error. Fixes #1553 Fixes BOT-Z4 Fixes BOT-Z8 Fixes BOT-Z9
\| \| \| * \| \| \| \| \| \| \|	CodeSnippets: avoid returning None when request raises an exception	MarkKoz	2021-04-28	-10/+13
\| \|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Move the exception handling to `on_message` to avoid writing a lot of None checks; `_fetch_response` is used multiple times in various places. Fixes #1554 Fixes BOT-Z7