-
Notifications
You must be signed in to change notification settings - Fork 28
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
slack build failed template nullpointerexception #44
Comments
Can I get some more detail on this? Like example config files. |
Here is this:
<?xml version="1.0" encoding="UTF-8"?>
<settings>
<webhooks enabled="true">
<webhook url="https://hooks.slack.com/services/xxxxxxx" enabled="true" format="jsontemplate" template="slack.com">
<states>
<state type="buildBroken" enabled="false" />
<state type="changesLoaded" enabled="false" />
<state type="buildInterrupted" enabled="false" />
<state type="buildFixed" enabled="false" />
<state type="buildFinished" enabled="true" />
<state type="buildStarted" enabled="false" />
<state type="responsibilityChanged" enabled="false" />
<state type="buildSuccessful" enabled="true" />
<state type="buildFailed" enabled="true" />
<state type="beforeBuildFinish" enabled="false" />
</states>
<build-types enabled-for-all="true" enabled-for-subprojects="true" />
</webhook>
</webhooks>
</settings> Are there any other config files that would be helpful? If it helps I'm on TC 10.0.1 |
Hmm, interesting. I've not played with branchSpec before. I'll leave this issue open, but low priority. |
I seem to have the same issue
|
Hi which version of the tcWebHooks plugin are you using? |
Also, @mmalvik, are you using branchSpec too? |
Oh, that branchSpec should be fine. I use that one all the time too. I probably should have asked if you have variables in the branch spec. I haven't done anything specific to fix this issue, but I would recommend updating as there are many many bug fixes and improvements. |
Btw. Looking at the stack trace, I'm not convinced you're seeing the same issue as Aaron was. Please do update, let me know if you see the issue again. |
I will keep an eye out for this and update if I see it again ✔️ |
Sorry, bumping this to 1.3 |
The text was updated successfully, but these errors were encountered: