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.

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

[Feature Request] Overhaul command handling system to utilize decorators instead of 'cmd_' method name prefixes #357

Open
DuckBoss opened this issue Jan 30, 2022 · 0 comments
Assignees
Labels
Feature Request A feature request High Priority A high priority feature/issue

Comments

@DuckBoss
Copy link
Owner

DuckBoss commented Jan 30, 2022

It might be a better idea to change the command handling system so that it uses a @command decorator instead of the current cmd_ prefix on methods. This would also open up the opportunity to add other custom decorators for other functionality.
For example:

OLD:
def cmd_echo(...):
    ...

NEW:
@command
def echo(...):
    ...
@DuckBoss DuckBoss self-assigned this Jan 30, 2022
@DuckBoss DuckBoss added Feature Request A feature request High Priority A high priority feature/issue labels Jan 30, 2022
@DuckBoss DuckBoss changed the title Overhaul command handling system to utilize decorators instead of 'cmd_' method name prefixes [Feature Request] Overhaul command handling system to utilize decorators instead of 'cmd_' method name prefixes Jan 30, 2022
@DuckBoss DuckBoss moved this from Being Investigated to Todo in JJMumbleBot v6.0.0 Major Release Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request A feature request High Priority A high priority feature/issue
Projects
Development

No branches or pull requests

1 participant