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

JenkinsPlugin not saving project level settings #34

Merged
merged 2 commits into from
Mar 12, 2019

Conversation

ScottBishop
Copy link
Contributor

@ScottBishop ScottBishop commented Mar 12, 2019

Plugins without project level settings were not saving properly because they were not submitted in the project edit form. I fixed this issue by using the PluginManager to loop through all the plugins explicitly when saving the Project form to ensure each plugins settings are saving correctly.

  • Fixed typo, and added another JenkinsPlugin test

@ScottBishop ScottBishop added the Needs Review Request a code review label Mar 12, 2019
Copy link

@andrewthomas91 andrewthomas91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andrewthomas91 andrewthomas91 added Ready To Merge Stamp of approval and removed Needs Review Request a code review labels Mar 12, 2019
@ScottBishop ScottBishop merged commit 48283ec into master Mar 12, 2019
@ScottBishop ScottBishop deleted the FixJenkinsPlugin branch April 1, 2020 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready To Merge Stamp of approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants