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

BUG: Running ./flow structureadjustments:fix doesn't mark workspaces as outdated #4508

Open
1 task done
ahaeslich opened this issue Sep 14, 2023 · 3 comments
Open
1 task done

Comments

@ahaeslich
Copy link
Member

ahaeslich commented Sep 14, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Running ./flow structureadjustments:fix will add your changes to the live workspace but not marking any of the other workspaces as outdated. Thus running ./flow workspace:rebaseoutdated afterwards will do nothing. Logging in again in a user workspace will rebase again.

Expected Behavior

Workspaces that doesn't contain the adjustments to be marked as outdated.

Steps To Reproduce

No response

Environment

- Flow: 9.0
- Neos: 9.0
- PHP: 8.2

Relates: #4442

@ahaeslich ahaeslich added the Bug label Sep 14, 2023
@ahaeslich
Copy link
Member Author

@skurfuerst in a slack conversation

ah. I think I understand why this is happening:
the "outdated" handling is in WorkspacesProjection::markDependentWorkspacesAsOutdated
=> called e.g. whenWorkspaceWasPublished.
=> when we directly publish new events on the live workspace, the system does not recognize that is has to mark the other workspaces as outdated.
I am not yet 100% sure how to fix this most cleanly. Either we emit a new event; or we somehow listen to all workspace changes (which I dislike)

@ahaeslich ahaeslich added the 9.0 label Sep 14, 2023
@bwaidelich
Copy link
Member

or we somehow listen to all workspace changes (which I dislike)

I actually think, that this is the way to go.
IMO we should keep "Publishing" as much out of the core logic as possible so that it is just a way to coordinate.
=> we would never really listen to publish/discard events (apart from directly affected projections) but to the actual node related events

@mhsdesign
Copy link
Member

mhsdesign commented Feb 7, 2024

Yes we noticed this problem as well when testing #4817

Logging in again in a user workspace will rebase again.

yes that was the behaviour but with #4865 it will be a noop like ./flow workspace:rebaseoutdated due to the check.

See also #4783

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

No branches or pull requests

3 participants