Filter by branch name

[doc-sheet]

Hello.

Is it possible to get notifications on specific branch only?

I tried to subscribe to pushes in master/devel branches and ignore other (usually WIP) pushes.

[thoratvinod]

Hello @hanzei, I am interested in this issue, may I take this up?

[hanzei]

Sure, thanks for taking a look @thoratvinod 👍

[thoratvinod]

Hi @hanzei ,

I have a question regarding the code. I've read through the codebase, and I noticed that when we run the command /github subscriptions add <repo>, it executes the handleSubscribesAdd method. However, I couldn't find the part in the code where we actually subscribe to the events. In the handleSubscribesAdd method, it seems like we are only adding subscriptions to the database as key-value pairs. Could you please let me know which file contains the code for subscribing to those events?

[hanzei]

Hi @hanzei,

Thanks for reaching out. Plugin.handleSubscribesAdd calls Plugin.Subscribe. That methods saves to subscriptions to the KV sore. The Plugin.handleWebhook method, which is called when a webhook event from GH comes in, the fetches the list of subscriptions from the store in e.g.

mattermost-plugin-github/server/plugin/webhook.go

Lines 732 to 735 in cbad10e

	func (p *Plugin) postIssueCommentEvent(event *github.IssueCommentEvent) {
	repo := event.GetRepo()
	subs := p.GetSubscribedChannelsForRepository(repo)

.

Please let me know if that helps.

[Kshitij-Katiyar]

Hi @thoratvinod, Do let us know if you have any other queries.