integrations / slack

Bring your code to the conversations you care about with the GitHub and Slack integration
https://slack.github.com/
MIT License
3.07k stars 483 forks source link

Update notification about accepted PRs after merge #1700

Open ad-m-ss opened 1 year ago

ad-m-ss commented 1 year ago

Is your feature request related to a problem? Please describe.

I would like to improve the reading of notifications to update the ones which are no longer relevant.

I would like leverage crossing out the notification when it's get expired. For example there is notifications which is crossed out, because I did review:

image

Describe the solution you'd like

I would like a cross-out notification about approved PRs when PRs are merged. I would like to update that notification:

image

Once PRs is merged, I think it's safe to assume that it's done, so all notification about that PRs can be cross-out.

Describe alternatives you've considered

We may also delete expired notifications, but I think it will be inconsistent and may be confusing.