integrations / slack

Bring your code to the conversations you care about with GitHub's integration for Slack
https://slack.github.com/
MIT License
3.09k stars 485 forks source link

Pull Request Reminders #789

Open arunpkio opened 5 years ago

arunpkio commented 5 years ago

It would be great If GitHub app can send reminders to the channel if there are Open Pull Requests which are still to be reviewed. There should be an option using which users can configure the interval for the reminders or remind the channel about all open PRs when a new PR is created or any existing PR is updated/closed.

welcome[bot] commented 5 years ago

Thanks for opening this issue! If you would like to help implement an improvement, read more about contributing and consider submitting a pull request.

neb commented 5 years ago

@arunpkqt See also https://github.com/integrations/slack/issues/380 from a year ago, including the commercial product mentioned in the comments. (I am not affiliated, but it is a pretty good implementation of exactly what you requested! And not too expensive.)

arunpkio commented 5 years ago

Thanks, @neb I have seen this but didn't give it a try. Will check.

abinoda commented 5 years ago

@neb Thanks for the shout-out! I maintain Pull Reminders 👋

If pricing is a problem for anyone reading this, please email me at abi@pullreminders.com and I'd be happy to give your team a free account.

stale[bot] commented 4 years ago

Is this still relevant? If so, just comment with any updates and we'll leave it open. Otherwise, if there is no further activity, it will be closed.

Dunedan commented 4 years ago

Yes it is still relevant. Pull Reminders got purchased by Github and creating new integrations with Pull Reminders doesn't work anymore (I always get some kind of redirect loop when trying to do so). On the other hand the Github Slack integration afaik doesn't support similar functionality yet.

radix commented 4 years ago

@pullreminders was acquired by github over 8 months ago, disabled signups, and there is still nothing to replace it. @abinoda you still handing out accounts?

JesusGVideoAmp commented 4 years ago

I'm also looking for alternatives here @abinoda

orj commented 4 years ago

GitHub has docs on setting up Pull Request Reminders but I can't find the option in my Company's Team settings.

https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/managing-scheduled-reminders-for-pull-requests

moraesdam commented 4 years ago

Followers, please check here #380

orj commented 4 years ago

As an update I figured out that you need opt in to the beta for pull reminders.

doublejosh commented 4 years ago

Another option https://github.com/ekmartin/slack-pull-reminder

arunpkio commented 4 years ago

Scheduled Reminders are now available from GitHub. More info can be found here https://github.blog/2020-04-21-stay-on-top-of-your-code-reviews-with-scheduled-reminders/

tommeier commented 4 years ago

Don't migrate to the new one yet - they don't seem to have a mapping of slack username to github username configurable anywhere (I regret migrating).

tobycox commented 4 years ago

@tommeier Yeah, I'm finding the same thing. Reached out to support a month ago and nothing...

chenbobby commented 4 years ago

@tommeier What's your use case? Do you have multiple slack username/workspaces and multiple github usernames?

I haven't used the new Scheduled Reminders yet but I'm guessing that they only support 1 github to 1 slack user...?

tommeier commented 4 years ago

@chenbobby we do have multiple - but this is scoped to one workspace - the current migrated implementation doesn't support mapping at all AFAIK.

pullreminders.com had a fantastic list of github users - option to add - then map to the slack username. Great, simple implementation. image

Our use case is simply that - a mapping of github users to one slack username for the current workspace the reminders are set to post in. I, wrongly, assumed that migrating to github reminders it would migrate this mapping. We'd become pretty reliant on pullreminders.com as it pinged everyone concerned and everyone cleared the deck before commencing their work day.

Choosing 'Manage reminder' takes you to https://github.com/organizations/XXXXX/settings/reminders/XXXX - which has a bunch of settings - but no github username to slack mapping.

There is some automatic mapping - I'm guessing based on username or slack 'friendly name' - which works some of the time. With around ~50 developers - i'm seeing about 10% of the notifications mapping to the slack username correctly. Even developers that map their exact github username as their slack name - no mapping.

Basically its near-useless without that mapping for us. Which is such a shame - it was fantastic before migrating.

chenbobby commented 4 years ago

@tommeier I think I'm getting what you're saying. It would definitely useful to map github users to slack usernames. This seems to a be a hot issue, according to #749 and #784 (also about linking github users to slack usernames).

I did some digging and found the /signin command, but it didnt seem to change anything in my bot notifications.

Screen Shot 2020-06-11 at 10 13 06 PM

It's exactly what you said:

Even developers that map their exact github username as their slack name - no mapping.

Maybe Github's "Scheduled Reminders" will get some updates soon for better integration.

btw, I'm also curious to know how PullReminders.com did their team organizations. Was the PullReminders.com account centered around a single Slack workspace, or around a single Github organization? Did people use PullReminders.com on their personal Github accounts, without an organization?

slmingol commented 4 years ago

I'd still like to see this feature integrated into the slack app. The scheduled reminders feature for our instances of repos in GitHub Enterprise have the Settings section of each team disabled for obvious reasons. By doing so it cuts you off from that feature completely.

image

As a non-admin on our GitHub account we cannot see this tab.

greveritt commented 4 years ago

I've been trying to get direct message reminders of review requests sent to a couple multi-channels guests in Slack. They were able to get such messages from Pull Reminders, but have not been able to get reminder messages since I migrated us to Scheduled Reminders. Can a full member of the Slack workspace set up the Slack integration to message the guests when needed?

The Slack documentation says that guests cannot connect to Slack integrations by themselves. Whenever our guests try to add scheduled reminders of their own, they end up seeing an error saying that Slack won't let them "install" the GitHub app to our Slack workspace, even though I already set up the Slack app myself and they have no need to "install" any app at all.

Thanks for listening to my concerns.

step 1

step 2

EDIT: It turns out they see this error when they try to invoke /github signin in Slack:

attempt to use :github signin

nabroyan commented 3 years ago

Any update on Github username to Slack username mapping?

tatane616 commented 3 years ago

https://github.com/integrations/slack/issues/789#issuecomment-664627329 I'm having the same problem, has anyone solved this?

jibsonline commented 3 years ago

+1

rogeryen commented 3 years ago

Also migrated over to scheduled reminders only to find out that it has stopped working completely for me. Sending a test message also doesn't work. Does this feature actually work for anyone?

jibsonline commented 3 years ago

@rogeryen that could be because the GitHub bot is not enabled on the channel you are trying to push notifications

rogeryen commented 3 years ago

@rogeryen that could be because the GitHub bot is not enabled on the channel you are trying to push notifications

Does it only work in channels? Because I'm really only interested in real time notifications, which I interpreted as DM from the @github bot?

jibsonline commented 3 years ago

Does it only work in channels? Because I'm really only interested in real time notifications, which I interpreted as DM from the @github bot?

Looking at the placeholder seems like it only works on channels image

You need to add @GitHub to the channel

rogeryen commented 3 years ago

Does it only work in channels? Because I'm really only interested in real time notifications, which I interpreted as DM from the @github bot?

Looking at the placeholder seems like it only works on channels image

You need to add @GitHub to the channel

Sucks if that's the case because pullreminders was working perfectly for me and I had to migrate myself over...

rogeryen commented 3 years ago

Does it only work in channels? Because I'm really only interested in real time notifications, which I interpreted as DM from the @github bot?

Looking at the placeholder seems like it only works on channels image You need to add @GitHub to the channel

Sucks if that's the case because pullreminders was working perfectly for me and I had to migrate myself over...

I did resolve the issue with suggestion from GitHub support to /github signout and /github signin again.

And also for the record, the “real time notification” feature is indeed DM from the GitHub bot.

jibsonline commented 3 years ago

@rogeryen so are you able to send it to yourself or still to a channel only?

rogeryen commented 3 years ago

@rogeryen so are you able to send it to yourself or still to a channel only?

@jibsonline yes I am able to get notifications from the bot to myself only in DM, when I enabled 'real time notification' in my personal settings.

trietbui85 commented 2 years ago

I notice the Pull Request Reminders still notify the approved PRs. 🤔 (for example in this case, it's 2 months old). Here is my configuration. image

Do you know what's wrong when I check the Ignore approved pull requests? I thought it's expected to ignore the approved PRs.

IGassmann commented 2 years ago

@anticafe same thing happening on my end. Have you figured how to fix this?

cosydney commented 2 years ago

I'm the creator of axolo. Axolo creates an ephemeral channel for each PR which is great for remembering the PR you have to review. On the reminder side, we have actionnable reminders who sends a message in this PR channel and tags the right person to act (reviewer, creator) depending on the status of the PR. You can reach out on our website happy to help.

rubengmurray commented 2 months ago

After a few false starts I've managed to get the integration working in the sense that it's linked my account and it's posted stale PRs to a Slack channel, but the app is only able to sync with certain repositories despite attempting the 'all repositories' and 'select repositories' options.

I've checked CODEOWNERS in repositories that are working and aren't working and things look identical so I'm not sure why some repos are available and others aren't 🤷

(Very happy to see the reminders for the ones that work though)