[JENKINS-70169] Add missing breadcrumb items in resources/hudson/model/Run
#7491
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Extract from #7457 (fixing the missing breadcrumb on /console view, the one at the origin of the Jira issue)
Follow-up of #6912
Initially fixing JENKINS-70169, I've included every missing breadcrumb items I've found by searching for "sidepanel.jelly" in jelly files.
Notes: This doesn't fix missing breadcrumb depending on plugin (ex: jenkinsci/cloudbees-folder-plugin#278)
Testing done
Built and ran Jenkins locally, going on modified pages to check the change.
There is no additional automatic test added for these breadcrumb items.
With a job "test",
http://localhost:8080/jenkins/job/test/1/configure
:http://localhost:8080/jenkins/job/test/1/confirmDelete
:http://localhost:8080/jenkins/job/test/1/console
:http://localhost:8080/jenkins/job/test/1/delete-retry
:I though core/src/main/resources/hudson/model/Run/artifacts-index.jelly would correspond to
localhost:8080/jenkins/job/test/1/artifact/
but I didn't managed to see my changes there, the corresponding change might be removed (?)Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@Wadeck @amuniz
Maintainer checklist
Before the changes are marked as
ready-for-merge
:upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).