Skip to content

Change LinksEnhancer to use Endpoint name #7132

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.

Sign up for GitHub

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

Jump to bottom
Closed
philwebb opened this issue Oct 8, 2016 · 0 comments
Closed

Change LinksEnhancer to use Endpoint name #7132

philwebb opened this issue Oct 8, 2016 · 0 comments
Assignees
@mbhave
Milestone
1.5.0 RC1

Comments

@philwebb
Copy link
Member

philwebb commented Oct 8, 2016

The current implementation of LinksEnhancer uses the path name for the rel. It would be better if we used names that were fixed so that if the path changes the rel stays the same.
@philwebb philwebb added this to the 1.5.0 milestone Oct 8, 2016
@mbhave mbhave modified the milestones: 1.5.0 M1, 1.5.0 Oct 11, 2016
@mbhave mbhave self-assigned this Oct 11, 2016
@philwebb philwebb mentioned this issue Oct 12, 2016
Closed
mbhave added a commit to mbhave/spring-boot that referenced this issue Oct 12, 2016
@mbhave
Change LinksEnhancer to user endpoint name
644509f 
If endpoint name is not available fallback to endpoint path.
Allow multiple hrefs per rel if path is different.

See spring-projectsgh-7132
mbhave added a commit to mbhave/spring-boot that referenced this issue Oct 12, 2016
@mbhave
Change LinksEnhancer to use endpoint name
6919bfb 
If endpoint name is not available fallback to endpoint path.
Allow multiple hrefs per rel if path is different.

See spring-projectsgh-7132
@mbhave mbhave mentioned this issue Oct 13, 2016
Closed
philwebb added a commit that referenced this issue Nov 16, 2016
@philwebb
Polish LinksEnhancer to use endpoint name
449b42f 
See gh-7164
See gh-7132
philwebb added a commit that referenced this issue Nov 16, 2016
@philwebb
Merge pull request #7164 from mbhave/gh-7132
4a6b49e 
* pr/7164:
  Polish LinksEnhancer to use endpoint name
  Change LinksEnhancer to use endpoint name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mbhave mbhave

Labels
None yet
Projects
None yet
Milestone
1.5.0 RC1
Development

No branches or pull requests
2 participants
@philwebb @mbhave