Feature request: Notify user who made commit

[niels-bosman]

Right now you can notify slack users for when the pipeline fails, would be nice if there was an option to map the GitHub users with the Slack username so only the person committing get's tagged in the message.

[ravsam-bot]

Hello and welcome to RavSam 💜. Thanks for opening your first issue. I will assign one of our team members to take a look at the problem you have raised. Make sure to include as much information as possible to help our team diagnose the issue and solve it quickly.

If the problem is related to a security concern, then please delete this issue and send us an email at info@ravsam.in.

[rdhar]

Related to #31.

[ravgeetdhillon]

@niels-bosman @rdhar Do you have any suggestions on how you would like this to be implemented from the end user perspective? What would be the format of the input that you would provide?

[rdhar]

To be honest, I'm not sure that a seamless implementation is possible between Slack user-mentions and GitHub username due to potential mismatch between user handles. That being said, I went through some of the most popular GitHub Action integration for Slack notifications to find a common theme:

• https://github.com/Ilshidur/action-slack uses GITHUB_ACTOR.
• https://github.com/voxmedia/github-action-slack-notify-build doesn't mention author.
• https://github.com/slackapi/slack-github-action doesn't mention author.
• https://github.com/8398a7/action-slack uses custom author field.
• https://github.com/rtCamp/action-slack-notify uses GITHUB_ACTOR.

Taking this into account, perhaps a new variable can be introduced to use GITHUB_ACTOR to notify in Slack.

• GITHUB_ACTOR is one of the default environment variables.
• github.actor is one of the github context properties.

[niels-bosman]

@ravgeetdhillon @rdhar I think it should be easy for users to enable this behaviour. The more complex logic should be dealt with by the package itself. Something like this:

steps:
  - uses: ravsamhq/notify-slack-action@v2
    if: always()
    with:
      status: ${{ job.status }}
      notify_actor: true # The new field
    env:
      SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }} # required

[ogind-bdhavale]

I am also facing similar issue. I am sending notification to slack channel and I want to mention user who has merged the PR or did the commit. The user is already part of slack channel.
The slack user name can be fetched by mapping of GitHub user email address and slack user email address.

[fede843]

+1

[jfoley15]

Also would love this enhancement.

@niels-bosman @rdhar Do you have any suggestions on how you would like this to be implemented from the end user perspective? What would be the format of the input that you would provide?

The input mention_users: "U0160UUNH8S,U0080UUAA9N" being able to accept other forms of id such as email address (available from the GitHub context) would be nice and easy for the end user imo