Skip to content

Changed linkTo so that it can be invoked outside of a HttpRequest. #410

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

Closed
wants to merge 2 commits into from

Conversation

nickgrealy
Copy link

Changed linkTo so that it can be invoked outside of a HttpRequest -
issue #408.
Added a linkTo method, which takes an explicit UriComponentBuilder.

I have signed and agree to the terms of the Spring Individual Contributor
License Agreement.

Changed linkTo so that it can be invoked outside of a HttpRequest -
issue spring-projects#408.
Added linkTo method, which takes an explicit UriComponentBuilder.
@odrotbohm odrotbohm force-pushed the master branch 2 times, most recently from 4ebc1be to 266ad50 Compare July 25, 2016 18:32
@gregturn
Copy link
Contributor

@nickgrealy This looks quite interesting. Do you think you could squash and rebase your PR against latest master?

@pivotal-issuemaster
Copy link

@nickgrealy Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@otrosien
Copy link

I like the idea of being able to use linkTo outside of the request context, but seriously, relying on catching an IllegalStateException to proceed with the program flow cannot be the solution to it...

gregturn added a commit that referenced this pull request May 25, 2017
If forming a link outside a Spring web request, fallback to relative links.

Original pull-request: #410
Related issues: #330, #143, #516
gregturn added a commit that referenced this pull request May 25, 2017
If forming a link outside a Spring web request, fallback to relative links.

Original pull-request: #410
Related issues: #330, #143, #516
gregturn added a commit that referenced this pull request May 25, 2017
If forming a link outside a Spring web request, fallback to relative links.

Original pull-request: #410
Related issues: #330, #143, #516
@gregturn gregturn self-assigned this May 25, 2017
gregturn added a commit that referenced this pull request May 25, 2017
If forming a link outside a Spring web request, fallback to relative links.

Original pull-request: #410
Related issues: #330, #143, #516
odrotbohm pushed a commit that referenced this pull request Jul 27, 2017
If forming a link outside a Spring web request, fallback to relative links.

Original pull-request: #410.
Related issues: #330, #143, #516.
odrotbohm added a commit that referenced this pull request Jul 27, 2017
Original pull-request: #410.
Related issues: #330, #143, #516.
@gregturn
Copy link
Contributor

Resolved via #408.

@gregturn gregturn closed this Aug 15, 2017
@gregturn gregturn added this to the 0.24 milestone Aug 15, 2017
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