Skip to content

Fix URL on images - #34

Merged
lvotypko merged 1 commit into
jenkinsci:masterfrom
ejouvin:master
Oct 12, 2015
Merged

Fix URL on images#34
lvotypko merged 1 commit into
jenkinsci:masterfrom
ejouvin:master

Conversation

@ejouvin

@ejouvin ejouvin commented Oct 1, 2015

Copy link
Copy Markdown
Contributor

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

ejouvin commented Oct 1, 2015

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Member

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

@KostyaSha

Copy link
Copy Markdown
Member

Looks like this bad test also failed other PRs

@walterk82

Copy link
Copy Markdown

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

@KostyaSha

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@glensc

glensc commented Mar 23, 2016

Copy link
Copy Markdown

@TobiX

TobiX commented Apr 15, 2016

Copy link
Copy Markdown

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

@glensc

glensc commented Apr 15, 2016

Copy link
Copy Markdown

oh. but what's holding the release?

@bluejaguar

bluejaguar commented May 25, 2016

Copy link
Copy Markdown

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

mibeumer commented Jun 9, 2016

Copy link
Copy Markdown

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

@smartmarmot

Copy link
Copy Markdown

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

@glensc

glensc commented Nov 22, 2016

Copy link
Copy Markdown

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

@smartmarmot

Copy link
Copy Markdown

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

@bluejaguar

Copy link
Copy Markdown

bump, for releasing .29 or higher

@fredg02

fredg02 commented Feb 17, 2017

Copy link
Copy Markdown
Member

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

@TobiX

TobiX commented Feb 17, 2017

Copy link
Copy Markdown

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

@fredg02

fredg02 commented Feb 21, 2017

Copy link
Copy Markdown
Member

@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.