| Commit message (Collapse) | Author | Age | Lines | 
| ...                        |  | 
| | |\ \ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / / /  
|/| | | | | | | | | | |    | 
 | 
| |\ \ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|_|_|/ / / / / /  
|/| | | | | | | | | | |    | 
Remove unwatch reason from the !nominate output
 | 
| |/ / / / / / / / / / /  
| | | | | | | | | | |   
| | | | | | | | | | |   
| | | | | | | | | | |    | 
As the watch reason can contain private information, we shouldn't share it with the whole staff.
 | 
| | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | |  | 
 | 
| | |\ \ \ \ \ \ \ \ \ \  
| | | |_|_|_|_|/ / / /  
| | |/| | | | | | | |   
| | | | | | | | | | |    | 
Since the cogs folder has been removed, the error_handler and tag cogs had to be removed and transfer into their respective places in the exts folder.
 | 
| | |\ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
https://github.com/python-discord/bot into feat/F4zi/CommandSuggestion
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
Co-Authored-By: Shirayuki Nekomata <[email protected]>
 | 
| | |/ / / / / / / / / /  
| | | | | | | | | | |   
| | | | | | | | | | |   
| | | | | | | | | | |    | 
After we send a tag or a suggestion we should return True for the error handler to know we handled the misspelt invoke
 | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | |  | 
If we sent a message, we need to return True,
the error_handler (in case it called the func)
will not send another message if already sent one.
 | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | |  | 
we cannot perform anything without the tags cog and commands
 | 
| | |\ \ \ \ \ \ \ \ \ \   | 
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
Since no tags were found, we can return false
since we didn't send any matching tag
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
If the code ran to the last line of the function without returning
(we return when we send a tag)
It means we didn't send the tag and need to return False
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
After CommandError was catched, the embed was sent,
now we return after the exception was catched
Modified the way raw_commands are collected
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
the CommandError exception was ignored,
We now catch the exception to handle and log the event
Also added a little code reformat
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
The message sent when suggesting a command is now wrapped inside an embed
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
get_command was changed to - display_tag, the name didn't fit,
since its not the command itself.
command_on_cooldown was taken out of display_tag to get the option to reuse it in another scope
Docstrings modified
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
The returned value was changed from True to False
If we didn't return True (sent the tag) throughout the code
It means we didn't send anything, we should return False
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
When found a similar command, the bot sent a suggestion about the full command name
This was fixed by saving the matching alias and sending it instead of sending the full command name
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
preventing message spam by the bot when commands are misspelt
 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | |  | 
Since its used to get the input of the command, its not needed again
In the function (callback) call
 | 
| | | | | | | | | | | | |      |  | 
| | |  \ \ \ \ \ \ \ \ \ \     |  | 
| | |\ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | |  | 
https://github.com/python-discord/bot into feat/F4zi/CommandSuggestion
 | 
| | | |\ \ \ \ \ \ \ \ \ \ \ \   | 
 | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | |  | 
Migration to error_handler.py
Suggesting misspelt commands, in progress
 | 
| | |/ / / / / / / / / / / / /  
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |    | 
Updated
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | |\ \ \ \ \ \ \ \  
| |_|_|_|_|_|/ / / / / / / /  
|/| | | | | | | | | | | | |    | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | |  | 
Annihilate all traces of Developer and Unverified roles
 | 
| |/ / / / / / / / / / / / / /   | 
 | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | |  | 
We have a check in place to restrict tag usage to a certain role, but our default is the Developers role, and some users now don't have this code.
This commit fixes this by using None as a default and adding a truth test in the check_accessibility method.
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | |  | 
To keep everything at one place, moved all branding manager special
things to one module.
 | 
| | | | | | | | | | | | | | |  | 
 | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | |  | 
Co-authored-by: Dennis Pham <[email protected]>
 |