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

fixing JCasC compatibility #8

Merged
merged 2 commits into from
Dec 13, 2018

Conversation

ewelinawilkosz
Copy link

As reported in kuleuven#31 I broke the plugin.
After all the fix was a little bit more complicated and I used PR by @timja for slack plugin as an inspiration: jenkinsci/slack-plugin#404

it seems you can configure global configuration both manually and with jcasc now
you can also manually configure what's in the job

What's missing

  1. Can't fill Notification message includes choice with proper options
  2. Haven't tested if it actually works with mattermost - I'm not using it and if anyone here can spare some time on verifying with that version that would be great, if not I'll try to do that later
  3. Haven't tested if it's working well with JobDSL (not sure if it could break but better safe than sorry)

So I'll continue my work to fix 1. But I wanted to share what I have so far, so anyone can contribute with some testing if you'd like

cc @jovandeginste

@jovandeginste
Copy link

jovandeginste commented Dec 10, 2018

@ewelinawilkosz Just tried your commit on our set-up, I can save the configuration

@ewelinawilkosz
Copy link
Author

thanks for checking, but I'm hitting the wall - anyone has an idea why doFillCommitInfoChoiceItems is not filling "Notification message includes" dropdown list?

@ewelinawilkosz
Copy link
Author

well... that's embarassing - f: was missing 😞

@ewelinawilkosz ewelinawilkosz changed the title [WiP] fixing JCasC compatibility fixing JCasC compatibility Dec 13, 2018
@jovandeginste
Copy link

@ewelinawilkosz deploying this change on our set-up, I will provide feedback

@jovandeginste
Copy link

@ewelinawilkosz seems to work, I will release this as a fix?

@jovandeginste jovandeginste merged commit a0dcda2 into jenkinsci:master Dec 13, 2018
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

Successfully merging this pull request may close these issues.

2 participants