Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add SlicedModel class for representing Slice collections #1856

Closed
wants to merge 1 commit into from

Conversation

mschout
Copy link
Contributor

@mschout mschout commented Oct 4, 2022

This is groundwork needed to support SlicedResourcesAssembler in spring-data-commons, by adding a SlicedModel class to spring hateoas. I have another PR coming which adds the SlicedResourcesAssembler support to spring-data-commons. Thi sis my first Spring contribution, so if I'm on the wrong track here, let me know.

See spring-projects/spring-data-commons#1307

This is groundwork needed to support SlicedResourcesAssembler<T> in spring-data-commons.

See spring-projects/spring-data-commons#1307
@mschout mschout force-pushed the feature/sliced-model branch from ab9fad3 to b25292e Compare October 4, 2022 22:59
@odrotbohm odrotbohm force-pushed the main branch 3 times, most recently from 2e02d7a to 856b6b9 Compare November 16, 2022 17:53
@odrotbohm odrotbohm self-assigned this Feb 28, 2023
odrotbohm pushed a commit that referenced this pull request Feb 28, 2023
In parallel to PagedModel we now provide a SlicedModel. This is groundwork needed to support SlicedResourcesAssembler<T> in spring-data-commons.

Related ticket: spring-projects/spring-data-commons#1307
odrotbohm added a commit that referenced this pull request Feb 28, 2023
Imports, formatting, Javadoc. Ticket references in test cases.
odrotbohm added a commit that referenced this pull request Feb 28, 2023
Add missing asciidoctorj-diagram dependency to render PlantUML diagrams correctly.
@odrotbohm odrotbohm added type: enhancement in: core Core parts of the project labels Feb 28, 2023
@odrotbohm odrotbohm added this to the 2.1.0-M1 milestone Feb 28, 2023
@odrotbohm
Copy link
Member

That's polished and merged. Thanks for your contribution!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
in: core Core parts of the project type: enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants