Skip to content
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

Replace ExpectedVersion::ANY() with proper version checks #5058

Open
bwaidelich opened this issue May 16, 2024 · 0 comments
Open

Replace ExpectedVersion::ANY() with proper version checks #5058

bwaidelich opened this issue May 16, 2024 · 0 comments
Labels

Comments

@bwaidelich
Copy link
Member

In the current core code we have 21 occurrences of ExpectedVersion::ANY().
Some of them are desired, for example in EventsToPublish::empty().

But all content-stream related events should always be published with the actual version of the respective CS (for example in SubtreeTagging::handleTagSubtree().

For workspace related events we should consider adding the workspace version to the read model, too, once those are integrated to the Content Graph projection (see #5038)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Prioritized 🔥
Development

No branches or pull requests

1 participant