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

	Commit message (Collapse)	Author	Age	Lines
...
\| \| * \| \| \| \| \| \| \| \|	(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.
\| * \| \| \| \| \| \|	Add custom exceptions for each error state.	Leon Sandøy	2020-05-28	-8/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The bot can get into trouble in three distinct ways: - It has no Bot instance - It has no namespace - It has no parent instance. These happen only if you're using it wrong. To make the test more precise, and to add a little bit more readability (RuntimeError could be anything!), we'll introduce some custom exceptions for these three states. This addresses a review comment by @aeros.
\| * \| \| \| \| \| \|	Make prefix consts private and more precise.	Leon Sandøy	2020-05-28	-9/+9
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Reduce complexity on some of the typestring stuff.	Leon Sandøy	2020-05-28	-31/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Refactor error messages in _to_typestring and _from_typestring to just print the prefix tuples instead of that custom error string. - Create a RedisKeyOrValue type to simplify some annotations. - Simplify partialmethod calls. - Make the signatures for _to_typestring and _from_typestring one-liners - Fix a typo in the errors.
\| * \| \| \| \| \| \|	Merge branch 'master' into redis_persistence	Leon Sandøy	2020-05-27	-80/+154
\| \|\ \ \ \ \ \ \ \| \|/ / / / / / / \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge pull request #961 from ↵	Sebastiaan Zeeff	2020-05-27	-88/+161
\|\ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|/ / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	python-discord/moderation_commands_in_modmail_category Permit moderation commands in ModMail category
\| * \| \| \| \| \| \|	Add some tests for `in_whitelist_check`.	Leon Sandøy	2020-05-27	-0/+48
\| \| \| \| \| \| \| \|