-
Notifications
You must be signed in to change notification settings - Fork 690
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
[JENKINS-59459] Revert both fields from being transient #400
Conversation
What's this change for? Is it to allow older versions of the plugin to use a template saved by a newer version? @varyvol |
@MRamonLeon precisely that. |
Fixes a backwards compatibility introduced in #329 for reference |
I included an automated test. By the way this is related with JENKINS-59460 too. |
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.
Thanks for the test @varyvol !
@res0nance , would you like to review the test as well?
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.
Kind of wonder how this will work once we work on reorganizing the configuration classes. Some sort of XStream feature maybe?
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.
Test looks good
@res0nance I think this can be merged. |
Merged thanks @varyvol |
@res0nance thanks! Do you know if there are plans to deliver a new release soon? I think delivering this and #387 is interesting. |
JENKINS-59459
Revert both fields from being transient so that they are exported and thus older version of the plugins are still able to import the template.
@dgarzon