search

CSUA / csua-backend

A backend for the CSUA interblags.
MIT License
21 stars 19 forks source link

Implements !verify #143

Closed dlzou closed 3 years ago

royh02 commented 3 years ago

I'm curious. Can we potentially make use of discord.ext.commands.Bot to clean up our current commands? It definitely beats a series of on_message checks and provides a default help command that documents the behaviours of each command better.

dlzou commented 3 years ago

@royh02 Interesting, looks like it descends from Client so it might be straightforward to switch. I'd like to know if there's a way to limit some commands to certain channels/DM. Would you be down to work on this at some point? That being said, I would still prefer to get this PR merged first (if you don't see any errors) so we can get back to verifying Discord users ASAP.

Edit: probably can check the channel within a command function.

dlzou commented 3 years ago

Tested in the CSUA-Test server. Just trying to set a good precedent.