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

BUGFIX: Remove Neos WorkspaceName #4534

Merged
merged 5 commits into from
Sep 28, 2023
Merged

Conversation

bwaidelich
Copy link
Member

@bwaidelich bwaidelich commented Sep 19, 2023

Fixes: #4533

Neos ui part neos/neos-ui#3624

@mhsdesign
Copy link
Member

❤️

Copy link
Member

@mhsdesign mhsdesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking care, i think we need to adjust the neos ui too.

Ahh i see you did this here : neos/neos-ui#3624 ❤️

@mhsdesign
Copy link
Member

mhsdesign commented Sep 19, 2023

Since we removed the odd logic, i think we can remove also this findByPrefix as there is no usage anymore:

@mhsdesign
Copy link
Member

So i ran the neos-ui e2e tests (with neos/neos-ui#3624) locally and everything seems to work so i would like to merge this task :D

@mhsdesign mhsdesign merged commit 81ff141 into 9.0 Sep 28, 2023
5 checks passed
@mhsdesign mhsdesign deleted the bugfix/4533-remove-neos-workspacename branch September 28, 2023 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get rid of Neos WorkspaceName
3 participants