You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now command parameters are just lame strings you can pass to display in the help command.
Params probably need to be expanded to objects with a few props such as:
name
description
mandatory (important!)
etc?
These can then be used in the help command.
What's mandatory all about?
This will help us do command parameter length validation, need to be careful with it though. It should be illegal to have any mandatory params after an optional one.
When checking required parameter length, we only need to care about the number of mandatory params, this could be set on the Command itself within the constructor (as could the previously mentioned validation).
The text was updated successfully, but these errors were encountered:
Right now command parameters are just lame strings you can pass to display in the help command.
Params probably need to be expanded to objects with a few props such as:
These can then be used in the help command.
What's mandatory all about?
This will help us do command parameter length validation, need to be careful with it though. It should be illegal to have any mandatory params after an optional one.
When checking required parameter length, we only need to care about the number of mandatory params, this could be set on the Command itself within the constructor (as could the previously mentioned validation).
The text was updated successfully, but these errors were encountered: