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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| \| * \| \| \| \| \| \| \| \|	(ModLog Tests): Created reason shortening tests for `send_log_message`.	ks129	2020-04-16	-0/+29
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Infraction Tests): Created reason shortening tests for ban and kick.	ks129	2020-04-16	-0/+54
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Test Helpers): Added `__ge__` function to `MockRole` for comparing.	ks129	2020-04-15	-0/+4
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Added removal of infraction in DB, when applying infraction ↵	ks129	2020-04-14	-18/+28
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	fail. Also don't send DM in this case.
\| \| \| * \| \| \| \| \| \| \| \|	(Scheduler): Removed empty line when expiration not specified in ↵	ks129	2020-04-14	-3/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`apply_infraction`.
\| \| \| * \| \| \| \| \| \| \| \|	(ModLog): Applied force embed description truncating in `send_log_message` ↵	ks129	2020-04-14	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	to avoid removing newlines.
\| \| \| * \| \| \| \| \| \| \| \|	(Big Brother): Added truncating reason.	ks129	2020-04-09	-2/+3
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(Watchchannel): Added footer shortening.	ks129	2020-04-09	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \|
\| \| \| * \| \| \| \| \| \| \| \|	(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.
\| * \| \| \| \| \| \|	Merge pull request #947 from python-discord/redis_persistence	Sebastiaan Zeeff	2020-05-28	-182/+1061
\| \|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	RedisCache - Data Persistence
\| \| * \| \| \| \| \| \|	Move the `self.redis_closed` into session create.	Leon Sandøy	2020-05-28	-1/+1
\| \| \| \| \| \| \| \| \|
\| \| * \| \| \| \| \| \|	Prevent a state where a coro could wait forever.	Leon Sandøy	2020-05-28	-2/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This addresses a review comment by @aeros.