Mattermost-plugin-github: How can I susbcribe to a whole organisation?

[cpitclaudel]

Hi!

Is there a way to subscribe a channel to all repos in a Github organization? I only found /github subscribe org/repo, which requires one command per repository (and does not add new repositories).

Thanks!

[jwilander]

Hi @cpitclaudel, currently you need to manually subscribe to each repository if you wanted to do this. I'm not opposed to adding a feature for this though. I've added the help wanted label in case someone wants to work on it

[benschuster788]

I'll take this one

[hanzei]

Submitted a PR to fix 2.

[dhadiseputro]

I'll work on this, from where benschuster788 left off. I am not familiar yet with the code base and this can be my 'hello world' project if you guys allow. Am I good to take his code and add the fixes needed in #53 and #56 ?

[jasonblais]

@dhadiseputro Go for it! If you have any questions working through it let us know. You're also open to sharing the approach you're planning to take for review 👍

[dhadiseputro]

I am planning to do something similar to benschuster.
The high level idea is, in SubscribeOrg I'll get all the repos in that org and do AddSubscription for each repo.

I have some questions:

• If we pass repo="" to Subscribe, we will get here https://github.com/mattermost/mattermost-plugin-github/blob/master/server/subscriptions.go#L84, but then ghOrg is not getting use anywhere after that. Also, repo will still be empty, how can we continue to do the AddSubscription at the end of the method?
• What is channelID?

[jasonblais]

pinging @crspeller or @aaronrothschild on the above

For channelID - that refers to the ID of a Mattermost channel. To find the ID of a channel, click the dropdown next to the channel name, then select "View Info". The ID is displayed at the bottom of the resulting dialog

[crspeller]

Looks like https://github.com/mattermost/mattermost-plugin-github/pull/67 already implements this?
Is there work left to do on this ticket? @jwilander

[dhadiseputro]

Ah, yes.. #67 implemented this ticket. So repo can have the value of either just the org's name or the complete org/repo right? I was a little confused with the var naming

[jwilander]

Yes, sorry should have closed this earlier. Sorry for the churn @dhadiseputro