Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send notifications to Microsoft Teams #91

Closed
harish123321 opened this issue Dec 29, 2018 · 9 comments
Closed

Send notifications to Microsoft Teams #91

harish123321 opened this issue Dec 29, 2018 · 9 comments

Comments

@harish123321
Copy link

harish123321 commented Dec 29, 2018

I want to send build information to Microsoft Teams from GOCD. Is there any way can I do it.

@ashwanthkumar
Copy link
Owner

@harish123321 We don't support Microsoft Teams right now. Let me know if you're willing to help by sending a PR. It should be easy to add that integration in here.

@sagrawal-myriad
Copy link

I was wondering if there is a push for this feature or if there is any other solution that could be used.

@ashwanthkumar
Copy link
Owner

I'm just going to put this out here, if someone is willing to help for this feature. You might need a parallel implementation of this class for MS Teams. I would be happy to review any PRs regarding this.

@lukelast
Copy link
Contributor

Hi @ashwanthkumar , I've started to work on this and would love some feedback if you have time.
https://github.com/lukelast/gocd-slack-build-notifier/commits/master

@jdavisp3
Copy link
Collaborator

Hi @ashwanthkumar , what do you think of the PR posted above?

@ashwanthkumar
Copy link
Owner

@jdavisp3 I did look at the commits on the fork and they look good. I haven't personally used Teams though so would need help in validating the changes in a real installation. For the changes to get merged, I would also need help with the documentation for the relevant bits.

@jdavisp3
Copy link
Collaborator

@jdavisp3 I did look at the commits on the fork and they look good. I haven't personally used Teams though so would need help in validating the changes in a real installation. For the changes to get merged, I would also need help with the documentation for the relevant bits.

🆒 we can help with that. We will be testing in a real installation and can help with documentation.

@lukelast
Copy link
Contributor

@ashwanthkumar I created a PR for Teams support.
#112

I took a stab at updating the readme. Please let me know if there's anything else I can do.

@ashwanthkumar
Copy link
Owner

Fixed via #112

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants