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

Show full display name for runs in promoted build parameters #116

Merged
merged 3 commits into from
Jun 4, 2018

Conversation

alhafoudh
Copy link
Contributor

Now promoted build parameter view displays full run name

Now promoted build parameter view displays full run name
@alhafoudh
Copy link
Contributor Author

In my workflow the build name is modified. It would be very handy to include the modified name as run parameter does

Copy link
Member

@dwnusbaum dwnusbaum left a comment

Choose a reason for hiding this comment

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

Sorry for not seeing this earlier, just have a question for clarification.

@@ -31,9 +31,9 @@ THE SOFTWARE.
<input type="hidden" name="name" value="${it.name}" />
<select name="runId">
<j:forEach var="run" items="${it.builds}">
<option value="${run.externalizableId}">${run}</option>
<option value="${run.externalizableId}">${run.fullDisplayName}</option>
Copy link
Member

Choose a reason for hiding this comment

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

Do you really want fullDisplayName, which will include all parent folders/jobs ("Folder » SubFolder » Job $BuildDisplayName"), or just displayName, which would be either "Job #NUM" or the build's modified display name in your case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I went with fullDisplayName because jenkins parameters use that. In fact I need modified display name :)

Copy link
Member

Choose a reason for hiding this comment

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

Ok, I will look around to see which seems more consistent across Jenkins, but other than that this PR seems totally fine to me, so I will try to merge+release it in the next week or so. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Glad to contribute :)

@dwnusbaum dwnusbaum changed the title Updated promoted build parameter view Show full display name for runs in promoted build parameters Jun 4, 2018
@dwnusbaum dwnusbaum merged commit eeee382 into jenkinsci:master Jun 4, 2018
@alhafoudh
Copy link
Contributor Author

Yay!

@dwnusbaum
Copy link
Member

@alhafoudh Thanks for the PR! It is included in version 3.2 which I just released and should be available via the update center within a few hours.

@alhafoudh
Copy link
Contributor Author

alhafoudh commented Jun 5, 2018 via email

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