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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| * \| \| \| \| \| \| \| \|	(Talent Pool): Applied reason shortening.	ks129	2020-04-09	-5/+5
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Replaced `infraction['reason']` with `reason` variable using in ↵	ks129	2020-04-09	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`end_msg`.
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Move reason to end of log text to avoid truncating keys.	ks129	2020-04-09	-2/+7
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Mod Utils): Moved embed description to variable.	ks129	2020-04-09	-5/+7
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Superstarify): Removed unnecessary truncation on `superstarify` command, ↵	ks129	2020-04-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	reordered ModLog text.
\| \| \| * \| \| \| \| \| \| \| \|	(Mod Utils): Removed truncation of reason itself and added truncation to ↵	ks129	2020-04-08	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	whole embed in `notify_infraction`.
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Removed reason truncation from `apply_infraction`, changed ↵	ks129	2020-04-08	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	order of ModLog embed description item in same function.
\| \| \| * \| \| \| \| \| \| \| \|	(ModLog): Added mod log item embed description truncating when it's too long.	ks129	2020-04-08	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Changed reason truncating in `apply_infraction` from 1900 chars ↵	ks129	2020-04-08	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	to 1500, added shortening to end message too.
\| \| \| * \| \| \| \| \| \| \| \|	(Superstarify, Scheduler): Added reason shortening for ModLog.	ks129	2020-04-08	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Infractions): Moved truncated reason to variable instead on ban coroutine ↵	ks129	2020-04-08	-5/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	creating.
\| \| \| * \| \| \| \| \| \| \| \|	(Infractions): Removed unnecessary logging that notify when reason will be ↵	ks129	2020-04-08	-6/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	truncated for Audit Log.
\| \| \| * \| \| \| \| \| \| \| \|	(Mod Utils): Added shortening reason on embed creation in `notify_infraction`.	ks129	2020-04-08	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Ban and Kick): Added space to `textwrap.shorten` `placeholder`.	ks129	2020-04-06	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Ban and Kick): Changed length in `textwrap.shorten` from 309 to 312 because ↵	ks129	2020-04-06	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	shorten already include `placeholder` to length.
\| \| \| * \| \| \| \| \| \| \| \|	(Mod Scheduler): Added reason truncations to Scheduler's `apply_infraction`	ks129	2020-04-05	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Ban and Kick): Applied simplification to reason truncating.	ks129	2020-04-05	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Ban and Kick): Replaced force reason truncating with `textwrap.shorten`.	ks129	2020-04-04	-2/+3
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Kick Command): Added logging and truncating to correct length for Discord ↵	ks129	2020-04-04	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Audit Log when kick reason length is more than 512 characters.
\| \| \| * \| \| \| \| \| \| \| \|	(Banning): Added logging and truncating to correct length for Discord Audit ↵	ks129	2020-04-04	-1/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Log when ban reason length is more than 512 characters.
\| \| \| \| \| \| \| \| * \| \| \|	Move counting of channels to a separate method	MarkKoz	2020-05-30	-10/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This de-clutters the main `server_info` function and improves its readability.
\| \| \| \| \| \| \| \| * \| \| \|	Simplify counting of staff channels and improve efficiency	MarkKoz	2020-05-30	-49/+26
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simplification comes from being able to access permissions as attributes on the overwrite object. This removes the need to iterate all permissions. Efficiency comes from checking all roles within a single iteration of all channels. This also removes the need to flatten and filter the channels afterwards, which required additional iterations.
\| \| \| \| \| \| \| \| * \| \| \|	Merge branch 'master' into staff_count_server	Leon Sandøy	2020-05-30	-1/+1
\| \| \| \| \| \| \| \| \|\ \ \ \ \| \| \|_\|_\|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Merge pull request #972 from Numerlor/tag-encoding	Mark	2020-05-29	-1/+1
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \| \|_\|_\|_\|_\|/ / / / / / \| \|/\| \| \| \| \| \| \| \| \| \|	Specify encoding when reading tag files
\| \| * \| \| \| \| \| \| \| \| \|	Tags: explicitly use UTF-8 to read files	MarkKoz	2020-05-30	-1/+1
\| \|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Not all operating systems use UTF-8 as the default encoding. For systems that don't, reading tag files with Unicode would cause an unhandled exception. (cherry picked from commit adc75ff9bbcf8b905bd78c78f253522ae5e42fc3)
\| \| \| \| \| \| \| * \| \| \|	Don't membership check in an itertools.chain.	Leon Sandøy	2020-05-30	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We're using the set comprehension to prevent duplicates anyway, so flipping these back makes more sense. Also added a missing ctx and tested ok.
\| \| \| \| \| \| \| * \| \| \|	Prevent duplicates, and break into function.	Leon Sandøy	2020-05-30	-27/+32
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- We're using a set comprehension and flipping the order for counting the number of channels that are both staff allow and @everyone deny. - We're breaking the staff channel count stuff into a separate helper function so it doesn't crowd the server_info() scope. These fixes are both to address the code review from @MarkKoz, thanks Mark.
\| \| \| \| \| \| \| * \| \| \|	Fix busted test_information test.	Leon Sandøy	2020-05-30	-4/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	I wish this test didn't exist.
\| \| \| \| \| \| \| * \| \| \|	More precise staff-channel check.	Leon Sandøy	2020-05-30	-13/+53
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We now check: - Does the @everyone role have explicit read deny permissions? - Do staff roles have explicit read allow permissions? If the answer to both of these are yes, it's a staff channel. By 'staff roles', I mean Helpers, Moderators or Admins.
\| \| \| \| \| \| \| * \| \| \|	Merge branch 'master' into staff_count_server	Leon Sandøy	2020-05-30	-0/+1
\| \| \| \| \| \| \| \|\ \ \ \ \| \| \|_\|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Merge pull request #965 from Akarys42/patch-1	Dennis Pham	2020-05-29	-0/+1
\| \|\ \ \ \ \ \ \ \ \ \ \| \| \|_\|_\|_\|/ / / / / / \| \|/\| \| \| \| \| \| \| \| \|	Ignore response when posting python news
\| \| * \| \| \| \| \| \| \| \|	Merge branch 'master' into patch-1	Dennis Pham	2020-05-29	-31/+8
\| \| \|\ \ \ \ \ \ \ \ \ \| \| \|/ / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \| \|	Ignore response when posting python news	Matteo Bertucci	2020-05-29	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Sometimes a mailing list user doesn't press respond correctly to the email, and so a response is sent as a separate thread. To keep only new threads in the channel, we need to ignore those.
\| \| \| \| \| \| \| * \| \| \|	Adding staff member count to !server.	Leon Sandøy	2020-05-30	-4/+10
\| \| \| \| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| * \| \| \|	Clean up channel counts and add staff channels.	Leon Sandøy	2020-05-29	-2/+22
\| \| \|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Cleaning up a particularly dirty line by turning it into like 10 lines, and also adding the number of channels that are hidden to the `@everyone` role - which we're classifying as "Staff channels".
\| * \| \| \| \| \| \| \| \|	Merge pull request #966 from python-discord/sebastiaan/help-channels/ratelimits	Joseph Banks	2020-05-29	-31/+8
\| \|\ \ \ \ \ \ \ \ \ \| \| \|/ / / / / / / / \| \|/\| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \| \|	Reduce the number of help channel name changes	Sebastiaan Zeeff	2020-05-29	-29/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Discord has introduced a new, strict rate limit for individual channel edits that reduces the number of allow channel name/channel topic changes to 2 per 10 minutes per channel. Unfortunately, our help channel system frequently goes over that rate limit as it edits the name and topic of a channel on all three "move" actions we have: to available, to occupied, and to dormant. In addition, our "unanswered" feature adds another channel name change on top of the move-related edits. That's why I've removed the topic/emoji changing features from the help channel system. This means we now have a generic topic that fits all three categories and no status emojis in the channel names.
\| \| * \| \| \| \| \| \| \|	Fix `check_for_answer` breaking on missing cache	Sebastiaan Zeeff	2020-05-29	-2/+5
\| \|/ / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `check_for_answer` method of the HelpChannels cog relies on the channel->claimant cache being available. However, as this cache is (currently) lost during bot restarts, this method may fail with a KeyError exception. I've used `dict.get` with an `if not claimant: return` to circumvent this issue.
\| \| \| \| \| \| * / /	Stats: Implement rules stats	ks129	2020-06-05	-0/+3
\| \| \|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|	Increase every shown rule uses count when command called.
\| \| \| \| \| \| \| *	Fix exception message assertions in converter tests	MarkKoz	2020-04-28	-7/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `msg` arg is for displaying a message when the assertion fails. To match against the exception's message, `assertRaisesRegex` must be used. Since all of the messages are meant to be interpreted literally rather than as regex, `re.escape` is used.
\| \| \| \| \| \| \| *	Use await instead of asyncio.run in converter tests	MarkKoz	2020-04-28	-28/+27
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Remove redundant parenthesis from test values	MarkKoz	2020-04-28	-19/+19
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Test for out of range datetime in the Duration converter	MarkKoz	2020-04-28	-0/+11
\| \| \| \| \| \| \| \|
\| \| \| \| \| \| \| *	Converters: handle ValueError when year for duration is out of range	MarkKoz	2020-04-25	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`datetime` objects only support a year up to 9999. Fixes #906
* \| \| \| \| \| \| \|	Filtering: Add user avatar thumbnail to nickname alert embed	ks129	2020-06-11	-1/+2
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Filtering: Use walrus for better looking of code	ks129	2020-06-11	-4/+2
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Filtering: Fix nickname filter alert sending spaces	ks129	2020-06-11	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Joseph Banks <[email protected]>
* \| \| \| \| \| \| \|	Filtering: Fix some comments	ks129	2020-06-11	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Co-authored-by: Joseph Banks <[email protected]>
* \| \| \| \| \| \| \|	Remove redis namespace collision test	MarkKoz	2020-05-31	-10/+0
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Remove redis namespace collision prevention	MarkKoz	2020-05-31	-10/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When cogs reload, it would consider their namespace as a conflict with the original namespace. This feature will be removed as a fix until we come up with a better solution.