search

ErythroGuild / irene

<Erythro>'s server admin bot.
Mozilla Public License 2.0
0 stars 0 forks source link

Add `CommandTree` constructor exclusively for context menu commands #405

Closed Ernest314 closed 1 year ago

Ernest314 commented 1 year ago

Should use a RootTree internally.

Having this separate would allow for removing the CommandType arguments from regular slash commands, and allow for setting description to "" in the context menu constructor.

The context menu constructor would still need a CommandType argument to differentiate between user and message context menus.

Maybe change all three to static factory methods? "with-single-command", "with-subcommand", "with-context-menu"

Ernest314 commented 1 year ago

Don't really need the complexity of factory methods; the overloads are clear enough.