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 support for slash commands #61

Open
twouters opened this issue Nov 16, 2018 · 1 comment
Open

Add support for slash commands #61

twouters opened this issue Nov 16, 2018 · 1 comment

Comments

@twouters
Copy link
Contributor

We can get a list of supported slash commands with commands.list on connect and listen for commands_changed RTM events to get updates.

Sending commands is possible too.

I'm not sure yet how we should expect the user to input the slash commands. Riot does not send unknown commands as messages, so either we expect users to escape the slash (\/<command> [args] ...) or we define a specific keyword to prepend slash commands (!slackcmd <command> [args] ...).

@thomas-profitt
Copy link
Member

I dig it.

I don't think we would want Slack commands in bridged Matrix rooms to look like Matrix commands anyway, when the convention in Matrix is for commands implemented by bots to be prefixed when ! instead of /, so I think something like "!who" or "!slack /who" or something would feel better than "/who" working as if it were a Matrix/Matrix client command.

Some Slack commands' output may need to be handled in special ways too, like the only-visible-to-me messages not being visible in Matrix (creating another issue for that now)

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

No branches or pull requests

2 participants