Skip to content

Improve documentation of entity state detection #951

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 3 commits into from

Conversation

schauder
Copy link
Contributor

The use of a version property to determine the state of an entity wasn't properly documented so far.

Related tickets spring-projects/spring-data-commons#2338

The use of a version property to determine the state of an entity wasn't properly documented so far.

Related tickets spring-projects/spring-data-commons#2338
@schauder schauder requested a review from mp911de March 26, 2021 09:03
@mp911de
Copy link
Member

mp911de commented Mar 26, 2021

It would make sense to extract this section into a fragment that can be referenced from Spring Data Commons in modules that apply IsNewStrategy.

@schauder
Copy link
Contributor Author

I'll do that.

@schauder schauder self-assigned this Apr 13, 2021
@schauder schauder assigned mp911de and unassigned schauder Apr 13, 2021
@mp911de mp911de changed the title Improve documentation of entity state detection. Improve documentation of entity state detection Apr 13, 2021
mp911de pushed a commit that referenced this pull request Apr 13, 2021
The use of a version property to determine the state of an entity wasn't properly documented so far.

Closes: #951.
mp911de added a commit that referenced this pull request Apr 13, 2021
Fix includes.

Original pull request: #951.
@mp911de mp911de closed this in 140c774 Apr 13, 2021
mp911de added a commit that referenced this pull request Apr 13, 2021
Fix includes.

Original pull request: #951.
@mp911de mp911de added this to the 2.1.8 (2020.0.8) milestone Apr 13, 2021
@mp911de mp911de deleted the issue/version-doc branch April 13, 2021 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants