mattermost / mattermost-plugin-github

GitHub plugin for Mattermost
Apache License 2.0
157 stars 148 forks source link

Support excluding specific comment authors for issue comment subscriptions #739

Open mickmister opened 7 months ago

mickmister commented 7 months ago

Comments from the codecov bot can be a bit spammy in channels subscribing to issue/PR comments. For my use case of the GitHub issue comments subscription, I would like to exclude the codecov bot from creating issue comments in the issue comment subscription.

The task here is to implement a --exclude-users flag that can take a comma-separated list of GitHub usernames to exclude from the subscription posts. This is currently implemented for DM notifications through the mute command, though is not accessible in the subscriptions feature

https://github.com/mattermost/mattermost-plugin-github/blob/2608197665120cdd99d5deae9fb01fe7bee6f20e/server/plugin/command.go#L216

hanzei commented 7 months ago

FYI: There is already a per-user setting in the form of /github mute. It seems useful to me to have both (a per-subscription and a per-user mute) :+1:

mickmister commented 7 months ago

@hanzei Funny enough, I had that in the issue description but removed it for some reason

Are you suggesting we call the flag mute when creating/editing the subscription?

mickmister commented 6 months ago

@hanzei I'd like to move this ticket to the monorepo to increase visibility. Do you have any thoughts on my question above?

hanzei commented 6 months ago

My concern is more about whether this setting should be per-user, per-subscription or global. There is already a per-user one. What value does a per-subscription setting add?

mickmister commented 5 months ago

@hanzei The per-user one is for DM notifications. How are you proposing this feature should work? I guess I'm simply not understanding what the exact suggestion is

mickmister commented 4 months ago

@hanzei Do you think this would work well? Or how do you see this working?

/github subscriptions add mattermost/mattermost-plugin-github --exclude-users codecov,mattermod