-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
TASK: Remove Neos.Node.inBackend()
and Neos.Node.isLive()
eel helper
#4542
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just wondered: do we want to keep Neos.Node.isLive()
or did we discuss to remove this Eel Helper as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes good point we want to remove the Neos.Node.isLive()
- i didnt know it existed ;)
... and if we remove |
It was deprecated in #4505 in favor of `${userInterfaceMode.isEdit}`. Since the ui is adjusted in neos/neos-ui#3618 it can now be removed.
68efe66
to
41651f6
Compare
Neos.Node.inBackend()
eel helperNeos.Node.inBackend()
and Neos.Node.isLive()
eel helper
Ah forgot that @mficzel already adjusted the rector migrations in neos/rector#22 🤦 I did adjust Neos.Seo where we introduced |
It was deprecated in #4505 in favor of
${renderingMode.isEdit}
. Since the ui is adjusted in neos/neos-ui#3618 it can now be removed.Relates: #4086
Checklist
FEATURE|TASK|BUGFIX
!!!
and have upgrade-instructions