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

Added /invite slash command #648

Merged
merged 3 commits into from
Sep 1, 2015
Merged

Added /invite slash command #648

merged 3 commits into from
Sep 1, 2015

Conversation

geekgonecrazy
Copy link
Contributor

Allow invite via: /invite username

@geekgonecrazy
Copy link
Contributor Author

This would close #592

@rodrigok
Copy link
Member

So nice 😸

Can I invite user with @, like /invite @rodrigo.rocket.team?

Could make sense cause we can use auto complete of mention popup.

@geekgonecrazy
Copy link
Contributor Author

heh no I didn't think of that. Shouldn't be too hard to add though. I'll try and add that real quick. :)

@geekgonecrazy
Copy link
Contributor Author

Actually.. not sure what's going on. Even checking for and removing the '@' right before. It doesn't seem to work. Weird..

@rodrigok
Copy link
Member

rodrigok commented Sep 1, 2015

If you comment the package to prevent automatically installation I can accept your PR and help you :)

@geekgonecrazy
Copy link
Contributor Author

Alright there we go. Commented out.

rodrigok added a commit that referenced this pull request Sep 1, 2015
@rodrigok rodrigok merged commit fe81ae5 into RocketChat:master Sep 1, 2015
@rodrigok
Copy link
Member

rodrigok commented Sep 1, 2015

@geekgonecrazy done :)

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

Successfully merging this pull request may close these issues.

2 participants