mattermost / mattermost-plugin-github

GitHub plugin for Mattermost

Mattermost GitHub Plugin

A GitHub plugin for Mattermost. Supports GitHub SaaS and Enterprise versions.

See the Mattermost Product Documentation for details on installing, configuring, enabling, and using this Mattermost integration.

Development

This plugin contains both a server and web app portion. Read our documentation about the Developer Workflow and Developer Setup for more information about developing and extending plugins.

Releasing new versions

The version of a plugin is determined at compile time, automatically populating a version field in the plugin manifest:

• If the current commit matches a tag, the version will match after stripping any leading v, e.g. 1.3.1.
• Otherwise, the version will combine the nearest tag with git rev-parse --short HEAD, e.g. 1.3.1+d06e53e1.
• If there is no version tag, an empty version will be combined with the short hash, e.g. 0.0.0+76081421.

To disable this behaviour, manually populate and maintain the version field.

How to Release

To trigger a release, follow these steps:

1. For Patch Release: Run the following command:

   make patch
   

   This will release a patch change.

2. For Minor Release: Run the following command:

   make minor
   

   This will release a minor change.

3. For Major Release: Run the following command:

   make major
   

   This will release a major change.

4. For Patch Release Candidate (RC): Run the following command:

   make patch-rc
   

   This will release a patch release candidate.

5. For Minor Release Candidate (RC): Run the following command:

   make minor-rc
   

   This will release a minor release candidate.

6. For Major Release Candidate (RC): Run the following command:

   make major-rc
   

   This will release a major release candidate.

Playwright e2e tests

In order to get your environment set up to run Playwright tests, please see the setup guide at e2e/playwright.

mattermost-plugin-github's Issues

Add "Attach to Github Issue" similar to Jira plugin

Example from the Mattermost Jira plugin:

See https://github.com/mattermost/mattermost-plugin-jira

https://mattermost.atlassian.net/browse/MM-18132

How can I susbcribe to a whole organisation?

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!

Connect one repo instead of an organization

Use case from customer:

Since we are connecting back to a Mattermost channel, and orgs can span many teams, being able to provide the hook at the repo level would be very helpful.

This might need an OAuth for each repo connection, but not sure.

Build a github action to Post to Mattermost

Similar or better to https://github.com/apex/actions/tree/master/slack

See https://github.com/actions for more info on github actions

Make access to KV store atomically safe when saving/removing subscriptions

Summary

For subscriptions of GitHub projects to MM channels, all of the subscription configuration is stored in the subscriptions portion of the KV store slice that is used by the Github plugin. In order to avoid race conditions and conflicts when two users are concurrently editing subscriptions, we should wrap the operations in a function like atomicModify used in the Jira plugin.

Ticket Link

https://mattermost.atlassian.net/browse/MM-18722

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Plugin: GitHub" community channel.

New contributors please see our Developer's Guide and our Plugins Guide.

"Unread messages" button links to wrong page

Clicking on the "Unread messages" button on the LHS open a list of every issue and PR where the user has been mentioned. But the number in the Mattermost UI shows the number of notification a user has. That's kind of confusing. I think the button should link to the GitHub notification screen. But I'm not sure how to filter the notification by organisation.

Plugin user profile image is missing

I have connect the plugin to github, but I found the user profile image can't load correct.

The image link is https://mattermost-server/api/v4/image?url=https%3A%2F%2Fassets-cdn.github.com%2Fimages%2Fmodules%2Flogos_page%2FGitHub-Mark.png, it return 403 error.
And I found GitHub-Mark.png is 404, can the plugin use my own profile image?

Show code block preview of Github code line permalink

When a user post contains a Github code line permalink, it should attach a markdown code block that will contain the preview of that section of the code with possibly 3 lines before and after (much like the display shown in code compares in Github:

The Github plugin should subscribe to the MessageWillBePosted hook and also include a setting to enable/disable this feature.

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18880

Make links to comments have the full comment link in the URL

Summary

Currently, a (username) commented on your pull request DM message from the Github bot will link to the associated pull request or issue, but will not include the full link to the comment. Because of this, the user is not taken directly to the comment when they click the link.

To improve UX of this feature, the user should be taken to the comment or review that caused the notification.

Ticket Link

https://mattermost.atlassian.net/browse/MM-19443

Fail to enable plugin

My mattermost server is 5.8.0, I start server with docker-compose and make plugins folder as volume by use - ./volumes/app/mattermost/plugins:/mattermost/plugins:rw

After I uploaded the plugin and set the config, I click the Enable button and got log below, plugin enable failed. Did I miss something?

{"level":"error","ts":1550461595.1755967,"caller":"app/plugin.go:97","msg":"Unable to activate plugin","plugin_id":"github","error":"unable to copy webapp bundle directory: github: mkdir client/plugins/github: permission denied","errorVerbose":"mkdir client/plugins/github: permission denied\nunable to copy webapp bundle directory: github\ngithub.com/mattermost/mattermost-server/plugin.(*Environment).Activate\n\t/go/src/github.com/mattermost/mattermost-server/plugin/environment.go:189\ngithub.com/mattermost/mattermost-server/app.(*App).SyncPluginsActiveState\n\t/go/src/github.com/mattermost/mattermost-server/app/plugin.go:95\ngithub.com/mattermost/mattermost-server/app.(*App).InitPlugins\n\t/go/src/github.com/mattermost/mattermost-server/app/plugin.go:126\ngithub.com/mattermost/mattermost-server/app.(*Server).RunOldAppInitalization.func6\n\t/go/src/github.com/mattermost/mattermost-server/app/server_app_adapters.go:142\ngithub.com/mattermost/mattermost-server/app.(*Server).InvokeConfigListeners\n\t/go/src/github.com/mattermost/mattermost-server/app/config.go:169\ngithub.com/mattermost/mattermost-server/app.(*Server).UpdateConfig\n\t/go/src/github.com/mattermost/mattermost-server/app/config.go:58\ngithub.com/mattermost/mattermost-server/app.(*App).UpdateConfig\n\t/go/src/github.com/mattermost/mattermost-server/app/config.go:62\ngithub.com/mattermost/mattermost-server/app.(*App).EnablePlugin\n\t/go/src/github.com/mattermost/mattermost-server/app/plugin.go:245\ngithub.com/mattermost/mattermost-server/api4.enablePlugin\n\t/go/src/github.com/mattermost/mattermost-server/api4/plugin.go:187\ngithub.com/mattermost/mattermost-server/web.Handler.ServeHTTP\n\t/go/src/github.com/mattermost/mattermost-server/web/handlers.go:143\ngithub.com/mattermost/mattermost-server/web.(*Handler).ServeHTTP\n\t<autogenerated>:1\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux.(*Router).ServeHTTP\n\t/go/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/mux/mux.go:162\ngithub.com/mattermost/mattermost-server/app.(*RateLimiter).RateLimitHandler.func1\n\t/go/src/github.com/mattermost/mattermost-server/app/ratelimit.go:107\nnet/http.HandlerFunc.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:1964\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers.recoveryHandler.ServeHTTP\n\t/go/src/github.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers/recovery.go:78\ngithub.com/mattermost/mattermost-server/vendor/github.com/gorilla/handlers.(*recoveryHandler).ServeHTTP\n\t<autogenerated>:1\nnet/http.serverHandler.ServeHTTP\n\t/usr/local/go/src/net/http/server.go:2741\nnet/http.(*conn).serve\n\t/usr/local/go/src/net/http/server.go:1847\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1333"}

The column on the left where the octocat icon shows up, is not visible for MM users that are only in 1 team.

the members of my dept. can't see the octocat icon, because they are only in one MM "team" and therefore don't have the strip on the left of the screen where the octocat icon shows up.

If I add them to a second MM "team" then they see the strip and the octocat

I don't know if this is a bug in that github plugin should cause that strip to show up, or if it is a bug in MM client.

Private repositories - working?

When I do

/github subscribe crudolf/XYZ

where XYZ is a private repository. The plugin tells me, that the repository cannot be found "Unknown repository .."

Can this plugin only handle public repositories?

octopus icon randomly changes where it points to

GitHub (github - 0.9.2)
GitHub plugin for Mattermost.

We are running on GithubEnterprise.

We have Enterprise Base URL and Enterprise Upload URL set to https://github.ourdomain.com

the octocat in the left bottom, sometimes points to https://github.com/settings/connections/applications/09a864219be324547cea1
and other times points to https://github.ourdomain.com/settings/connections/applications/09a864219be324547cea1

I can't get it to change, it just does what it wants. Currently, all the icons down there point to github.com and therefore don't work.

Make the timezone configurable

My whole team is in Europe/Zurich, but the timestamps in the plugin messages are always UTC.
It'd be nice if the timezone used in these messages could be configured (or if they were adapted to the local timezone of whoever is viewing it).

Update github.com/google/go-github dependency

Copied over from #74 (comment)

[Help Wanted] Add a command to list current channel's subscriptions

Add a command /github subscribe list that will list all the repository subscriptions of the current channel as an ephemeral message. It should also list the events that are subscribed to (e.g pulls,issues,labels etc.)

If you have questions feel free to ask here or join the GitHub Plugin channel on our community Mattermost server.

Support for both github.com and enterprise server

It is currently impossible to integrate two GitHub servers with the same plugin.

As a workaround: will stuff break if I edit the .tar.gz to just change plugin id?

Do not trigger mentions from plugin messages

It would be nice if the plugin messages did not trigger mentions (or if that could be configured by each user). When I merge a PR, I really don't need a notification on my mobile device about that...

Clicking on a LHS buttons shows non mattermost issues

Clicking on a button brings up non mattermost related PRs and issues. It is kind of confusing to see a different number of PRs when clicking on then button that show on the LHS.

If a user deletes a comment, you receive a notification for a comment creation

Description:
When a user is deleting comments on a PR e.g., you will be notified that the user posted a new comment:

Compared to:
mattermost/mattermost-webapp#2356

The "Comment created" from @jasonblais are actually comment deletes.

Actual Behavior:
Comment deletion being shown as comment creation

Expected Behavior:
Comment deletion being shown as comment deletion

Make prerequisites

make dist fails without the following "go get"s

go get github.com/google/go-github/github
go get github.com/mattermost/mattermost-server
go get golang.org/x/oauth2

If a user edits a comment with a mention of you, you get a second notification

Repro steps:

1. Mention a user, who has their Mattermost account connected with the plugin, in a GitHub issue/PR comment
2. Edit that comment (making sure to leave the mention in)

Observed:
Two notifications are sent to the user about the comment

Expected:
Only one notification is sent

Github Plugin: Add comment message in DM Notifications

The Github plugin sends DMs for Github notifications when someone has commented on your PR that currently looks like:

This DM should be improved to include the comment message like it currently does for mentions notifications DMs as:

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18877

Add Refresh ability to RHS

The GitHub plugin RHS lists notifications, pull requests, assigned issues and review requests. It currently displays the data that is available from the current state of the store but does not have the ability to refresh the data in case is outdated. Add support to Refresh the data by:

1. Calling Refresh whenever the RHS bar is opened to guarantee latest data is available.
2. Add an action icon on the title bar to offer the ability to manually refresh as needed (similar to the one currently available in the team sidebar buttons). Locate the icon in the RHS section title right aligned (in screenshot below, in the “Your Open Pull Requests” title header).

Current RHS:

Sidebar Buttons:

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18879

Improved refresh of RHS

The GitHub plugin RHS lists notifications, pull requests, assigned issues and review requests. It currently refreshes on demand, or whenever the GitHub plugin happens to receive an update that forces the client to fetch the latest results.

At this time, not every update is proxied to the GitHub plugin and thus some items that might be expected to disappear instantly – e.g. read notifications, or finishes reviews – stay in the interface longer than might be expected.

Let's introduce improved refresh semantics:

• keep track of when the browser gains/loses focus, perhaps by extending the plugin API to do this, or if we already track this in Redux, surfacing a selector to expose this to the plugin
• if the RHS is open, and the browser regains focus, trigger an automatic update on the assumption the user came back to the Mattermost application and wants new results.
• some items may disappear when the update occurs, but this might be jarring in the user interface: instead, keep these in memory within the component, and render any item not found in the canonical dataset in a dimmed/greyed out fashion (This is getting worked on in #138)

For notifications specifically let's also automatically dim/grey out items when clicked, since it's expected that these items will be marked as unread when the link is clicked, and the user might just be opening in new tabs without leaving Mattermost.

The net results of this might look like:

and it will now be easier for the enduser to click through their TODO list and know exactly where to pick up when they come back.

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18888

Use proper bot account

Mattermost 5.10 now allows bot accounts, and having a proper bot account instead of having to make a separate account for the bot would be nice.

Do not post a daily reminder if there are no updates

When there are no updates, the plugin sends a daily reminder with "no updates".

Propose not sending a daily reminder when there are no updates.

Empty slash command trows error

Running just /github return Received invalid response from the server. in the Mattermost interface.

Include a redirect_uri in the authorize request

Currently this is not the case, so I need to set the correct URI on the GitHub app registration. But this is a bad idea because without looking at the code I have no way of knowing what the exact redirect URI needs to be (https://.../plugins/github/oauth/complete)!

Github enterprise support

Is on-prem Github enterprise supported, is it possible to subscribe to github events based on mattermost channels, userids like userA wants to send all github events from OrgA to channel A.

Add "Create Github Issue" similar to Jira plugin

Example from the Mattermost Jira plugin:

See https://github.com/mattermost/mattermost-plugin-jira

https://mattermost.atlassian.net/browse/MM-18132

Use custom post type to display TODO list

Currently, the TODO list that gets posted daily or by /github todo looks something like this:

This works ok but it's hard to read, is only minimally functional and I think we can add something that does way better. Plugins have a cool feature that lets you use custom post types to render your own React components in place of the regular post components. Leveraging this we can:

1. Display way more information about the pull requests/issues upfront in a much more readable way
2. Add more functionality, such as a button to open all the unread messages so users don't have to click through each one

I started working on this in the unreads-v2 branch with the following commit 54e8ecf but stopped for two reasons: 1) I ran out of time and 2) plugins could not use custom post types for ephemeral messages (which the /github todo command returns). As of the next Mattermost version 5.12 coming out soon, 2) is possible thanks to mattermost/mattermost-webapp#2759. Unfortunately, 1) is still the case as I'm still a bit too busy to work on it.

If someone is interested in picking up where I left off I think this would be a really great feature that would take the GitHub plugin to the next level. It'll require a bit of everything from server side, front end, UX design and merging master into my branch.

Here's what it was looking like when I stopped:

Provide a more clear error message when the wrong content type is selected for a webhook

Repro:

1. Follow these steps to configure the GitHub plugin: https://github.com/mattermost/mattermost-plugin-github/#configuration

• NOTE: To reproduce the error in this issue, when creating the webhook in GitHub in step 2, do not change the content type to "application/json"

3. Connect your Mattermost account to GitHub with /github connect.
4. Subscribe a channel to receive notifications with /github subscribe owner/repo.
5. Trigger an event such as opening an issue in the GitHub repository you subscribed.

Observed: No event, with the following error in server logs

2019-06-17T13:35:18.794Z    info    go-plugin/stream.go:15    {"level":"error","msg":"invalid character 'p' looking for beginning of value"}    {"plugin_id": "github", "source": "plugin_stdout"}

Proposed change:

1. Update the server log error message to state GitHub webhook content type should be set to "application/json"
2. Fail the request with a proper status code so that the webhook shows up as failed on GitHub

userprofile hover github url is pointing to github.com

When this plugin is enabled, and you check someones userprofile in mattermost. There is github information if github plugin is enabled in that user. However, that url will always point to github.com instead of correct github what is defined (in our case github enterprise url)

Cannot link Github account via the Mattermost Mac desktop app

Trying to link github account via the Mattermost Mac desktop app leads to a popup window where the authorize button cannot be clicked. Trying to link the github account via the Mattermost Webapp(Chrome) is successful.

Subscribe repo to commits

Right now, it's possible to subscribe a repo to issues and PR's. It would be cool if it could also provide notifications for commits (like on slack).

Improve RHS UI and content

The GitHub plugin RHS lists notifications, pull requests, assigned issues and review requests. We would like to improve the content and UI layout to include:

• PR number
• Milestone version - if available
• Time the PR has been open - in the case of PR lists

The UI should look like:

PR Lists:

Issues List:

Icons used:

• fa-code-fork
• fa-bullseye for milestone

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-17962

RHS behavious badly if there are too many label on a PR

When a PR has too manly labels the RHS looks and behaves weird:

• The title is broken into two line
• The last label is cur off
• You can scroll to the right to show the last label
  

Don't send a notification for self assigend issues

When assigning an issue to yourself, you get a notification via DM.

This should not happen.

Document all slash commands under "Features" section of README

Improve "Attach to Issue" function

This was just completed by @niklabh #120

Comments from @DHaussermann are quoted below.

@aaronrothschild I do see some opportunities to add to this further that would improve the experience. Let me know of any of this would be worth adding and I can create separate issues.

1. When I attach a comment, I should get feedback that this was done successfully (Similar integrations post a reply with such a message and a link but, a simpler way to provide feedback wold work as well)

Yes, I've had a few instances where the confirmation is re-assuring - certainly during the first experience. I agree with @DHaussermann , the utility of having the link referenced within the conversation is pretty critical to the long-term user experience. Over time, without the reference to the attached issue, a critical piece of development conversations can get lost. it also serves the dual purpose of confirming that it was successful in attaching to the issue. I'm 4/5 we should add this before merge.

2. In the search results, ideally we would return the issue number in addition to the title to cover cases where there titles are similar. Though the issue numbers could overlap across multiple repos within the same list.

This request has come up from a few different customers that use the equivalent function in the Jira plugin. Displaying the issue numbers clears ambiquity and lets the user be confident they chose the right option/value. Not sure how much effort is involved.

3. This design is limited to adding comments where the user is the assignee. There would also be value in adding comments to other issues as well. Would we need a more complex design to handle cases where you select an issue and don'y have permissions to comment?

Hmmm. Not sure about this one, could we even accomodate a scenario where we couldn't post as the user (b/c permissions)?

Make fails

Running just make fails with:

make: *** No rule to make target '.npminstall', needed by 'check-style'. Stop.

I think, https://github.com/mattermost/mattermost-plugin-github/blob/master/Makefile#L8 should be changed to check-style: webapp/.npminstall gofmt

Allow excluding organisation members from subscription events

There should be an option to not post issue and PR events from organisation members. If the flag --exclude-org-member is provided when subscribing to a repo, events from organisation members will not trigger notification.

If not organisation is set (https://github.com/mattermost/mattermost-plugin-github/blob/master/plugin.json#L45-L48), this flag has no effect.

Compute label foreground colour using WCAG rules

The pending update to the GitHub plugin supports a RHS that renders labels on PRs:

Unfortunately, sometimes GitHub renders different foreground colours for the given label colour:

Investigate rtsao/gh-label-svg#2 and following the WCAG algorithm proposed there to converge on the right white or black foreground colour. If we can find the authoritative algorithm, use that instead.

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-19019

Link on RHS are not limited to organisation

When clicking on the of the links on the RHS, a new windows opens up in github. But the PRs/issues shown there are not limited to the organisation.

Plugin failed to ServeHTTP

Running on 5.3.1 fails with this error:

{"level":"error","ts":1539313420.9237213,"caller":"plugin/client_rpc.go:154","msg":"RPC call to OnActivate plugin failed.","plugin_id":"github","error":"reading body gob: name not registered for interface: "*plugin.ErrorString""}
{"level":"error","ts":1539313421.156202,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313421.16836,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313421.187666,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313421.194268,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313421.2369359,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"f}
{"level":"error","ts":1539313421.244325,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313421.322338,"caller":"plugin/client_rpc.go:295","msg":"Plugin failed to ServeHTTP, RPC call failed","plugin_id":"github","error":"connection is shut down"}
{"level":"error","ts":1539313426.156944,"caller":"plugin/client_rpc.go:268","msg":"Plugin failed to ServeHTTP, muxBroker couldn't Accept request body connecion","plugin_id":"github","error":"timeout waiting for accept"}
{"level":"info","ts":1539313436.7515883,"caller":"utils/i18n.go:83","msg":"Loaded system translations for 'en' from '/opt/mattermost/i18n/en.json'"}

Add PR Title in Notifications DMs

The Github plugin sends DMs for Github notifications related to PRs, i.e. when your PR is approved or when you are requested a review on a PR. The current format of the DM is:

These type of DMs should be improve to show the PR title as the text of the link to the PR. You can check out the text templates defined for the plugin’s webhooks here

---

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18878

Link directly to issues in subscriptions

Right now, the bot mentions the issue number for comments (e.g [#302 Issue Found]) but doesn't actually link this text to it. Would be incredibly helpful to do so.

Similar issue: closing issues links it, but doesn't show the issue number on the link.

server\Subscriptions.go: SubscribeOrg adding object as key to the subscription dictionary instead of repo full name

server\Subscriptions.go:
SubscribeOrg adding object as key to the subscription dictionary instead of repo full name .
I tracked the issue down to lines 125 and 128. After applying these changes the pluging worked as desired when adding subscriptions by organization.

       sub := &Subscription{
		ChannelID:  channelID,
		CreatorID:  userId,
		Features:   features,
		Repository: fmt.Sprintf("%s/%s", owner, repo),
	}
	if err := p.AddSubscription(fmt.Sprintf("%s/%s", owner, repo), sub); err != nil {
		continue
	}

Needs to be converted to

         sub := &Subscription{
		ChannelID:  channelID,
		CreatorID:  userId,
		Features:   features,
		Repository: repo.GetFullName(),
	}
	if err := p.AddSubscription(sub.Repository, sub); err != nil {
		continue
	}

The plugin does not send me any message

Hi All,

I have configured everything on our enterprise github and mattermost environment (both launched internally) using the link below:

https://github.com/mattermost/mattermost-plugin-github

but i am not getting notified for any changes on repos in my organization. Please let me know if you need any logs to troubleshoot the issue.
---Ajay