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: Using "Publish all changes to live" in the workspace modul throws error #4591

Closed
1 task done
ahaeslich opened this issue Oct 10, 2023 · 3 comments
Closed
1 task done
Assignees

Comments

@ahaeslich
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

While testing #4588 we found this behaviour while there was just 1 user logged in:

Selecting all nodes by using "Select all current changes" and click on " Publish selected changes to live" (Calls publishOrDiscardNodesAction) works. ✅

Selecting nothing and click on "Publish all changes to live" (Calls publishWorkspaceAction) will lead to this error message:

The base workspace has been modified in the meantime; please rebase.

Edit: ... sometimes 🤯

Expected Behavior

Publishing via content module and workspace module works similar.

Steps To Reproduce

  1. Edit a page or content
  2. Create a new page with subpages and e.g. content
  3. Publish only the new parent page via "Publish to Live" button
  4. Review all other changes in the workspace module (If your neos instance doesn't contain BUGFIX: Show and apply all changes in workspace review #4588 you need more than just new pages to get access to the discard/publish buttons)
  5. Try to publish your all changes via "Publish all changes to live" (aka don't select the checkboxes)

Result: The base workspace has been modified in the meantime; please rebase.

  1. Go to the content module
  2. Click "Publish all"
  3. Everything is published

Environment

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

Anything else?

No response

@ahaeslich ahaeslich changed the title BUG: <title> BUG: Using "Publish all changes to live" in the workspace modul throws error Oct 10, 2023
@mhsdesign mhsdesign moved this from Todo to Prioritized 🔥 in Neos 9.0 Release Board Oct 16, 2023
@kitsunet
Copy link
Member

kitsunet commented Nov 3, 2023

I currently can't reproduce this anymore? I guess sometimes and I just didn't find that some time yet?

@ahaeslich
Copy link
Member Author

ahaeslich commented Nov 3, 2023

I currently can't reproduce this anymore? I guess sometimes and I just didn't find that some time yet?

Yes that was the confusing part. Will try to reproduce it and otherwise close this issue for now ... Thx for the reminder

@ahaeslich
Copy link
Member Author

Can't reproduce it anymore. Therefore, we can close this issue.

@github-project-automation github-project-automation bot moved this from Prioritized 🔥 to Done ✅ in Neos 9.0 Release Board Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants