Skip to content

fixes-27350#48

Merged
undera merged 1 commit intojenkinsci:masterfrom
hitesh22:fixes-27350
Jul 6, 2016
Merged

fixes-27350#48
undera merged 1 commit intojenkinsci:masterfrom
hitesh22:fixes-27350

Conversation

@hitesh22
Copy link
Contributor

Add shorturi method back to UriReport. This method is used in Jelly script which displays the uri used for test.

Removed sample values need to be added back for the uriReport page.
URI column is redundant on the uriReport page.

Add shorturi method back to UriReport. This method is used in Jelly script which displays the uri used for test.

Removed sample values need to be added back for the uriReport page.
URI column is redundant on the uriReport page.
@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@jugalps
Copy link

jugalps commented Apr 8, 2015

I verified by manually installing the plugin from the build, however it does not populate the URI field:
image

@hitesh22
Copy link
Contributor Author

@jugalps I removed the URI column so I am not sure why the URI column is showing. What page are you looking at?

I am looking at the uriReport page: ../119/performance/uriReport/some-test.jtl%3Asome_uri.endperformanceparameter/

@manolo manolo self-assigned this Apr 21, 2015
@manolo manolo added the ready label Apr 21, 2015
@Gurvan-MycomOsi
Copy link

@manolo any chance to have this fix released soon ?

@esaborg
Copy link

esaborg commented Sep 16, 2015

Is there any progress with this? Would really appreciate if this fix would be released as binary.

@foragerr
Copy link

Any updates on when this PR would be merged?

@Zeloslaw
Copy link

Zeloslaw commented Jul 5, 2016

Please add
report.setReportFileName(reportFile.getName());
to AbstractParser.java line 67

@undera undera merged commit 6c746a8 into jenkinsci:master Jul 6, 2016
@hitesh22 hitesh22 deleted the fixes-27350 branch July 7, 2016 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants