dnh-bot / dnh-werewolf-bot

DNH Werewolf Discord bot
The Unlicense
11 stars 7 forks source link

Refactor parse_command #140

Closed neihousaigaai closed 8 months ago

huybt22 commented 8 months ago

Could you please elaborate about the flow checking valid channels?

I see you try to make the dynamic logic, however most of the handles are directly based on the function name strings or variable names placed in config.

Please make sure when we make changes to the string in texts or function/variable names, the features still work properly without modifying at too many other places.

Please add comments to any complex text string handlers for better readable and maintainable.

A possible use case is when the common keywords like LOBBY, GAMEPLAY, PERSONAL become HALL, VILLAGE, PLAYER.

Or the actions in do_actionname may change :D