How can I susbcribe to a whole organisation?

[mattermost]

How can I susbcribe to a whole organisation?

[benschuster788]

I'll take this one

[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

[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]

If anyone is interested working on this ticket:

1. First PR merged here: #52
2. Second PR containing further bug fixes was closed, but not merged, here: #56

Moreover, a PR to unsubscribe from an organization was closed but not merged, here: #53

[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 #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