-
Notifications
You must be signed in to change notification settings - Fork 16
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
fixes #72 #73
fixes #72 #73
Conversation
use correct ContentType to avoid 400 error
@Eric-Maddox I will gladly create a release for this, but the pipeline is not currently building, can you take a look? |
I don't know why the build is failing. If I create an empty folder, and do a |
Hi! We are using this plugin and are now seeing the exact same issue on our TeamCity instance. Could the broken build be fixed by bumping the Java version in the GitHub workflow config? It seems to currently be configured for Java 8 ( |
@Eric-Maddox Can you please sync with the latest commit on |
upgrade to Java 11
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@Eric-Maddox / @olamartensson -> This is merged and I created a new release (2.13.0). |
Thanks a lot! I have now tested the new release on our TeamCity instance and Teams notifications are working again 👍 |
fix IOException after MS changed response
use correct ContentType to avoid 400 error
Includes unit test.