Verification rewrite (#52 Hacktoberfest) - Githubissues

SharkyTheKing / Sharky

Sharky's Cogs. If you have any suggestions or issues, please feel free to add them here.

19 stars 8 forks source link

Verification rewrite (#52 Hacktoberfest) #55

Closed madebylydia closed 4 years ago

madebylydia commented 4 years ago

Preda told me that git's description isn't made for being a patch note... So guess what I did? I made a patch note for what I made.

General:

Remove Optional and import humanize_list and pagify.
Use commands.Cog instead of a getattr.
Added super().__init__().
Changed the config structure so we can add the new functionality.

All commands:

Added commands.Context to ctx argument.
Docstrings and strings: Special message to Sharky The King: DOTS, SHARKY, YOU KNOW WHAT A DOT IS?

verifyset:

Reduce config call.
Now indicate more informations.

verifyset active:

Now check if role (Either autorole/temprole) has been added before activating. Will autodisable if role configuration has been removed.
Config usage: Special message to Sharky The King: YOU KNOW YOU CAN DO THIS SUPER EASILY WITHOUT USING TOO MUCH IFIFIFIF???

verifyset temprole:

Stop getting always "Cleared the role being used" and show the help if there's nothing registered.
discord.AllowedMentions overwrite. In case I nuke Red one day.

verifyset autorole:

A nice place for autorole, time to trash Trusty's autorole cog. You can add, remove, and list roles.

agree:

Added manage_messages check.
Add verify alias. (I want though on this one)
Less config call.
Now just not only remove a role. :smile:
Added status field, to know if user received role(s). ctrl+c/ctrl+v is nice btw

SharkyTheKing commented 4 years ago

Thank you for this! Happy for this to look how it is, I have some ideas in the future which I'll happily run by you.