-
-
Notifications
You must be signed in to change notification settings - Fork 750
Move aliases to their respective cogs #707
Copy link
Copy link
Closed
Labels
a: backendRelated to internal functionality and utilities (error_handler, logging, security, utils and core)Related to internal functionality and utilities (error_handler, logging, security, utils and core)p: 3 - lowLow PriorityLow Prioritys: WIPWork In ProgressWork In Progresst: featureNew feature or requestNew feature or request
Metadata
Metadata
Assignees
Labels
a: backendRelated to internal functionality and utilities (error_handler, logging, security, utils and core)Related to internal functionality and utilities (error_handler, logging, security, utils and core)p: 3 - lowLow PriorityLow Prioritys: WIPWork In ProgressWork In Progresst: featureNew feature or requestNew feature or request
Type
Fields
Give feedbackNo fields configured for issues without a type.
I think it's easier to maintain aliases if they are located near the actual commands. Furthermore, creating separate functions for aliases feels redundant and harder to maintain (particularly the parameters and their type annotations). I wonder if it's possible to create a decorator for commands which can add an alias for a command. Perhaps the command resolution of discord.py can be manipulated rather than having to create "dummy" commands for aliases (this may prove to be complicated for subcommands/groups particularly).
The point of this issue right now is to have some discussion on what a good design would be and some ideas for implementations.