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

Allow multiple commands per message #46

Open
Sinthorion opened this issue Jul 27, 2018 · 0 comments
Open

Allow multiple commands per message #46

Sinthorion opened this issue Jul 27, 2018 · 0 comments
Labels

Comments

@Sinthorion
Copy link
Contributor

Sometimes a user writes multiple commands as one command per line in a single message, expecting the bot to understand it as separate commands. Maybe we could split the input text on all newlines and treat each line as its own command.

Example:

<User>:
.iam Merchants
.iam Español
<Ducky Mc Duckerson>:
Cannot find any role by the name 'Merchants
.iam Español'

Issues:

  • Potential performance loss, because split is a more demanding operation than checking for a single prefix
    => Split should only be applied after a prefix was found at the start of the message
  • Reactions to the message (such as the checkmark for success) are ambiguous
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant