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

Give karma to multiple users at the same time #38

Open
atuttle opened this issue Feb 25, 2016 · 1 comment
Open

Give karma to multiple users at the same time #38

atuttle opened this issue Feb 25, 2016 · 1 comment

Comments

@atuttle
Copy link
Collaborator

atuttle commented Feb 25, 2016

Not sure what the syntax should be... Maybe this?

++ @ryanguill @atuttle @danskaggs
@ryanguill
Copy link
Owner

I think what we will need to do is notice the ++ and the loop over the message.parts (which is the message split on spaces) and check each part if it is a username, keeping an array until we run out of usernames. Because you could do something like ++ user user1 user2 thanks for the help!

but if you do ++ user badusername user2 it will stop at user and not pick up user2 or the bad username.

also need to combine the confirmation messages I think.

The existing pattern match here: https://github.com/ryanguill/watney/blob/master/plugins/karma.js#L364 which is /^\+1[:,\s]+.+/g I think will pick this up, but there might be a better one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants