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

Enable some form of Slack notifications parsing #586

Closed
wuub opened this issue Dec 21, 2016 · 6 comments
Closed

Enable some form of Slack notifications parsing #586

wuub opened this issue Dec 21, 2016 · 6 comments

Comments

@wuub
Copy link

wuub commented Dec 21, 2016

Hi,

right now mentioning slack users or teams is pretty tedious as it has to be done with slack IDs. Think
<!subteam^ID|team> or <!@U12345|nickname> instead of normal @team or @nickname.

This is because slack does not parse messages received from webhooks (default parse=none).
I think parse=full is unnecessary, but adding link_names=1 here

type slackReq struct {
should be helpful.

What do you think? Are there any reasons why this is not there or will you accept a simple PR? :)

@juliusv
Copy link
Member

juliusv commented Dec 21, 2016

I think that makes sense. I only wonder whether it should be configurable (or always on), but probably configurable.

@zerthimon
Copy link

is this gonna be merged any time soon ?

@stuartnelson3
Copy link
Contributor

Following up now, sorry for letting this slip

@KirillTemnov
Copy link

We have same issue, can you fix it?

@brian-brazil
Copy link
Contributor

#912 added this.

@PaulAnnekov
Copy link

Looks like nowadays link_names: true + mentioning like @username stopped working. I tried even <@username> - no luck, it just prints this text as is. The only solution is to use user id, like <@USERID>. You can find user id using this manual https://help.workast.com/hc/en-us/articles/360027461274-How-to-find-a-Slack-user-ID.
I think this change is related to the blog post https://api.slack.com/changelog/2017-09-the-one-about-usernames.

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