-
Notifications
You must be signed in to change notification settings - Fork 245
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
Update 'Add credentials' dropdown button #551
Conversation
Has compatability with the ATH been checked with this? The code change otherwise looks reasonable to me. |
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.
@janfaracik This results in 36 new ATH failures in jenkinsci/acceptance-test-harness#1752. Can you prepare an ATH PR that adapts to this change? |
Thanks Basil! I will do :) |
@basil Heya, when updating ATH tests for plugin changes, do we need to maintain compatibility with the old version of the plugin? |
FWIW I know Basil asked you to maintain forward compatibility it depends on whether the plugin baseline is supported on the latest LTS version really. If the change will be released and it will work on latest weekly and latest LTS then you don't really need forward compatibility as we only support the latest version of the plugin in ATH, but if its easy to maintain the compatibility then it makes it so you can merge the ATH change independently which is a good thing |
This PR updates the 'Add credentials' button to use the
.jenkins-button
class as well as to use the Dropdown component over the legacy YUI dropdown.Before
After
Testing done
Submitter checklist