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.

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

Add a combined dialog prompt for all weapon attack inputs #56

Closed
kmoschcau opened this issue Oct 2, 2021 · 1 comment
Closed

Add a combined dialog prompt for all weapon attack inputs #56

kmoschcau opened this issue Oct 2, 2021 · 1 comment
Labels
enhancement New feature or request system something relating to the system itself

Comments

@kmoschcau
Copy link
Collaborator

kmoschcau commented Oct 2, 2021

Only prompt for missing information.
For example, when attack is executed from inventory, no need to prompt for skill. When a target is selected, no need to prompt for range.

Weapon attack prompts should be dialog boxes, not chat messages. The chat is only for the results.

@kmoschcau kmoschcau added enhancement New feature or request system something relating to the system itself labels Oct 2, 2021
@kmoschcau
Copy link
Collaborator Author

This can be made even more generic, by passing an object with metadata to the prompt, having it generate a form dialog according to that and return the object with filled data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request system something relating to the system itself
Projects
None yet
Development

No branches or pull requests

1 participant