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

Fix URL on images #34

Merged
merged 1 commit into from
Oct 12, 2015
Merged

Fix URL on images #34

merged 1 commit into from
Oct 12, 2015

Conversation

ejouvin
Copy link
Contributor

@ejouvin ejouvin commented Oct 1, 2015

When display a project,, icons before Job, All bilds, Locked builds, All
workspaces, Slave workspaces, Non-slave workspaces have a broken link.

The root URL was missing, add ${resURL} as prefix in the attribute src,
for node img.

When display a project,, icons before Job, All bilds, Locked builds, All
workspaces, Slave workspaces, Non-slave workspaces have a broken link.

The root URL was missing, add ${resURL} as prefix in the attribute src,
for node img.
@ejouvin
Copy link
Contributor Author

ejouvin commented Oct 1, 2015

How a change on image URL can failed to build.
Sorry if I did something wrong. But it seems same tests previously failed and were success after.

@KostyaSha KostyaSha closed this Oct 1, 2015
@KostyaSha KostyaSha reopened this Oct 1, 2015
@KostyaSha
Copy link
Member

@ejouvin test can be unreliable, retriggered pr, let's look.

@KostyaSha
Copy link
Member

Looks like this bad test also failed other PRs

@walterk82
Copy link

I applied the SNAPSHOT build for this and it works fine when using /jenkins/ as the prefix on my installation.

@KostyaSha
Copy link
Member

@lvotypko highlight

lvotypko added a commit that referenced this pull request Oct 12, 2015
@lvotypko lvotypko merged commit 05dd253 into jenkinsci:master Oct 12, 2015
@thomasgl-orange
Copy link
Contributor

FYI, there is a Jira bug report that this PR fixes: JENKINS-30830
https://issues.jenkins-ci.org/browse/JENKINS-30830

@glensc
Copy link

glensc commented Mar 23, 2016

in what version this was fixed? i have 0.28 and i still have broken images on job details page because the urls are absoulte from site root. my jenkins installation is prefixed with /jenkins

accesses: https://jenkins.example.org/plugin/disk-usage/icons/directory16.png - 404
must access: https://jenkins.example.org/jenins/plugin/disk-usage/icons/directory16.png - 200

also in ticketing system there are still two tickets about same problem still open:
https://issues.jenkins-ci.org/browse/JENKINS-33083
https://issues.jenkins-ci.org/browse/JENKINS-30830

@TobiX
Copy link

TobiX commented Apr 15, 2016

@glensc This isn't in the 0.28 release. There wasn't a release yet after this was fixed.

@glensc
Copy link

glensc commented Apr 15, 2016

oh. but what's holding the release?

@bluejaguar
Copy link

bluejaguar commented May 25, 2016

Ok, I ran into this issue during install of Jenkins 2.5. Had to pull/clone and build locally myself to get 0.29-snapshot. Why on earth has this not been released yet?

@mibeumer
Copy link

mibeumer commented Jun 9, 2016

I also has this icon error. Please release a new version

@smartmarmot
Copy link

Same issue is happening to me, has this issue been solved?

@glensc
Copy link

glensc commented Nov 22, 2016

if it happened today then likely no release has been made, include your version here to be sure.

@smartmarmot
Copy link

Just checked and the plugin is: disk-usage plugin 0.28

@bluejaguar
Copy link

bump, for releasing .29 or higher

@fredg02
Copy link
Member

fredg02 commented Feb 17, 2017

@lvotypko please release a new version of the plugin so everyone can get this fix.

@TobiX
Copy link

TobiX commented Feb 17, 2017

This plugin looks abandoned. Last commit was over a year ago...

@fredg02
Copy link
Member

fredg02 commented Feb 21, 2017

I've posted about this on the Jenkinsci-dev mailing list: https://groups.google.com/forum/#!topic/jenkinsci-dev/JO3ONqRlP3o

@froque froque mentioned this pull request Jan 28, 2023
6 tasks
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.