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

[FIXED JENKINS-16207] TFS 2012+ Links Fixed #26

Closed
wants to merge 9 commits into from

Conversation

DD202
Copy link

@DD202 DD202 commented Sep 19, 2014

Fixed the merge issues with the last request. Option exists under the Adanced section to use the REST style urls created with TFS 2012+

@jenkinsadmin
Copy link
Member

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

@olivierdagenais olivierdagenais self-assigned this Sep 25, 2014
@olivierdagenais
Copy link
Member

Sorry for the very long delay; when I first got commit access to this project, I had been only tackling my immediate needs at $DAYJOB.

I am planning to review these changes for release 3.3.0.

@mosabua
Copy link
Member

mosabua commented Mar 16, 2015

I would love to get this one into a release soon! +1

@mosabua
Copy link
Member

mosabua commented Feb 10, 2016

@olivierdagenais I was just checking the repo browser URL persistence on a real server and found that we need this merged for links to changesets and so on to actually work. Are you mostly okay with how this PR is implemented? If yes, I will clean it up into a separate branch and then test and merge it.

@mosabua
Copy link
Member

mosabua commented Feb 10, 2016

Also might default the new parameter to true so that it will work out of the box for new TFS server installations..

@mosabua
Copy link
Member

mosabua commented Feb 10, 2016

Given that we do not even support older versions out of the box I think this might be overkill. Instead of a config flag and so on we might want to just change the URL to use the new URLs. Can you confirm that these style URLs are used in all current versions? I will prepare a PR for this approach next.

@mosabua
Copy link
Member

mosabua commented Feb 11, 2016

Replaced by #71

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.

4 participants