slack icon indicating copy to clipboard operation
slack copied to clipboard

Slack notifications not working on mentions in comments on commits

Open Birajpjpt opened this issue 4 years ago • 23 comments

Describe the bug When a user is mentioned in the comments in any commits, they aren't being notified over the slack. They are however notified via email.

To Reproduce Steps to reproduce the behavior:

  1. Go to a new or any other commit
  2. Scroll all the way down and in write the comment section, add any comment while mentioning a user.
  3. The user isn't notified over slack for the mention

Expected behavior Any user mentions should've resulted in a slack notification real time.You are mentioned in a comment option within enable real time alerts is also properly checked.

Birajpjpt avatar Jul 13 '21 08:07 Birajpjpt

@Birajpjpt I think its only supported for the mention in the comment on the pr. Keeping this for feature request.

amitkumariiit avatar Aug 12 '21 07:08 amitkumariiit

This should work for normal issue comments as well, or the wording should be updated to explain it is specifically a PR comment. It reads like any @mentions will cause a github notification currently. To clarify I'm looking for comments on normal issues, not on comments on commits.

Note: I'm experiencing this issue in the Github Reminders feature, not the github slack app. This might be the wrong PR for that?

IronSean avatar Sep 08 '21 18:09 IronSean

when someone update existing comment - new notifications about those update not appeared in defined slack channel that is subscribed to comments

user216 avatar Aug 01 '22 16:08 user216

+1 to get real-time alerts for all mentions, including on issues and commits Also +1 that an update to an existing comment (to PR, issue, or commit) should trigger a new alert

andnor avatar Oct 27 '22 20:10 andnor

+1 to slack notifications for issues -- my organization barely uses email and slack integration would make collaboration much faster

aHagouel avatar Dec 05 '22 18:12 aHagouel

any update on this? Has anyone figured out how to receive Slack notifications for issues (not prs)?

TziviElias avatar Jan 02 '23 09:01 TziviElias

I use Zapier to create a zap to cover this missing functionality https://zapier.com/shared/1e8b590be3b8d18979d1d5491b8cc8d71494e418

daxmc99 avatar Jan 03 '23 20:01 daxmc99

+1 on this feature being fixed!

noahsidney avatar Mar 07 '23 16:03 noahsidney

any update? according to the documentation this should work so this is actually a bug?

https://github.com/integrations/slack#mentions

angalic avatar May 25 '23 08:05 angalic

a duplicate of this issue: #1669

angalic avatar May 25 '23 08:05 angalic

Same issue with commits to a branch -- no notifications. This seems like a bug, but guessing same issue as here with comments.

tomharrisonjr avatar Aug 18 '23 19:08 tomharrisonjr

This used to work, but no longer. Perhaps GitHub has changed its API and broken the integration with Slack?

johnyaku avatar Nov 13 '23 08:11 johnyaku

I have the same thing. We get 0 notifications on comments on PRs, inline comments and comments on commits.

noudadrichem avatar Nov 27 '23 15:11 noudadrichem

+1. I'm missing out on mentions in PRs. what's going on?

letiesperon avatar Dec 08 '23 17:12 letiesperon

I get no notifications on anything. Tried to roll this out to the team with no luck.

keithdoyle9 avatar Dec 13 '23 20:12 keithdoyle9

As this is a serious blocker of using this plugin, isn't anyone interested to address this :thinking:

cwill-smg avatar Jan 23 '24 10:01 cwill-smg

As this is a serious blocker of using this plugin, isn't anyone interested to address this 🤔

Hi @cwill-smg, yes we are!! check us out at https://axolo.co. We have an opinionated way of dealing with a GitHub - Slack integration, we do a two-way sync and create an ephemeral channel for each pull request. Let us know what you think!

cosydney avatar Jan 23 '24 10:01 cosydney

My team of 8 is having this issue too.

trezum avatar Apr 26 '24 09:04 trezum

Same here guys, any updates? @ashokirla

redac avatar May 03 '24 15:05 redac