Add support for link-names=1

[vasicvuk]

It should be nice to have an option to enable link_names when sending webhook request. This will enable tagging people or channels.

https://api.slack.com/incoming-webhooks

It is simple parameter in json that is sent.

[tboerger]

Sorry, but can you explain what you are talking about? I don't get what you are requesting.

[vasicvuk]

Slack webhooks gives option to mention someone In messages with @username. This currently is not working if you put @username In template. This is enabled by adding list_names: 1 In JSON payload.

Sample JSON:
{"text":"Hello @myname", "link_names": 1 }

This way we could get ability to post to channel and mention user created the commit

[donny-dont]

@vasicvuk have you looked at slack-blamer? It actually uses the slack api rather than a webhook and is much more fully featured.

[vasicvuk]

Yes but i thought that this is deprecated plugin or so because link to the Docs is not working from GitHub repo. And also it is not listed on plugins.drone.io.

On the other side link_names is Incoming webhook feature not API feature.

[tboerger]

Next time you should mention link_names instead of list_names and clearly explain what you want to archive, than somebody else will understand it directly and can suggest a solution.

A pull request to add this option is welcome.

[donny-dont]

slack-blame isn't deprecated. We use it quite a bit internally. It might not have been updated because we internally are on an ancient version of Drone

[bradrydzewski]

@vasicvuk I am going to close this issue. Please send a pull request if you want to see it implemented