Skip to content

#288 - Opt to render single links as an array. #588

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
wants to merge 1 commit into from
Closed

#288 - Opt to render single links as an array. #588

wants to merge 1 commit into from

Conversation

gregturn
Copy link
Contributor

@gregturn gregturn commented May 22, 2017 •
edited
Loading

Provide the means to render a single link entry as a JSON Array.

Original pull-request: #295
Related issues: #291, #115

@gregturn gregturn requested a review from odrotbohm May 22, 2017 20:24
@gregturn gregturn self-assigned this May 22, 2017
@odrotbohm
Copy link
Member

What if we allowed users to rather register a bean HalConfiguration or the like — which we'd of course default in our configuration — on which they can configure HAL-specific settings. That link relation and multiplicity issue seems very HAL specific which makes me think it's to specific to justify a top level configuration property in @EnableHypermediaSupport. Looks decent beyond that to me.

@gregturn
#288 - Opt to render single links as an array.
7cd1fb8 
Provide the means to render a single link entry as a JSON Array.

Original pull-request: #295
Related issues: #291
@gregturn gregturn force-pushed the feature/single-link-as-array branch from 1acafc5 to 7cd1fb8 Compare September 11, 2017 15:21
@odrotbohm odrotbohm closed this Oct 13, 2017
@odrotbohm odrotbohm deleted the feature/single-link-as-array branch October 13, 2017 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@odrotbohm odrotbohm Awaiting requested review from odrotbohm

Assignees

@gregturn gregturn

Labels
process: in progress type: enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gregturn @odrotbohm