Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add decorator to require server usage #178

Open
andrewj-brown opened this issue Nov 8, 2023 · 0 comments
Open

Add decorator to require server usage #178

andrewj-brown opened this issue Nov 8, 2023 · 0 comments
Labels
enhancement update an existing command or cog for some new functionality

Comments

@andrewj-brown
Copy link
Member

Several commands do permissions checks that fall over or behave annoyingly when they're used in DMs. (PR #176) Several more commands just generally assume they occur in servers, and may fail in spectacular ways if they aren't.

It'd be nice if we could codify this with some sort of @require_server decorator, that requires the interaction to take place inside UQCSBot.uqcs_server.
@andrewj-brown andrewj-brown added the enhancement update an existing command or cog for some new functionality label Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement update an existing command or cog for some new functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andrewj-brown