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

Created new command model #121

Merged
merged 6 commits into from
Sep 26, 2017
Merged

Created new command model #121

merged 6 commits into from
Sep 26, 2017

Conversation

purrplingcat
Copy link
Owner

@purrplingcat purrplingcat commented Sep 25, 2017 •
edited
Loading

For support to resolve #58

Created new command model as OOP ES6 classes.

  • Commander (command registry)
  • Command (Base class of commands)
  • GroupCommand (commands with subcommands)
  • SimpleCommand (simple class for hot create a command)
  • BuiltinCommands (static clas for register base commands)
  • CommandMessage (represents an message from discord with command argv)
  • CommandArgv (command argv from called command)

This model is compatible with old solution.

It's not fully integrated to bot! Will be integrate step-by-step 🐱

@purrplingcat purrplingcat mentioned this pull request Sep 26, 2017
Closed
@purrplingcat purrplingcat merged commit fcbc4b9 into master Sep 26, 2017
@purrplingcat purrplingcat added this to the 1.4.0-rc milestone Sep 30, 2017
@purrplingcat purrplingcat deleted the commandmodel branch October 25, 2017 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
semantic:PullRequest type:Feature type:Refactor
Projects
None yet
Milestone
1.4.0-rc
Development

Successfully merging this pull request may close these issues.

Prefixes (scopes) for commands - Native subcommands support
1 participant
@purrplingcat