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: Position TemplateNodeCreationHandler to the end #46

Merged
merged 2 commits into from
Jul 22, 2023

Conversation

mhsdesign
Copy link
Contributor

@mhsdesign mhsdesign commented May 31, 2023

also we dont replace the original one anymore

closes #45

BLOCKED BY neos/neos-ui#3511

@theilm
Copy link
Collaborator

theilm commented May 31, 2023

Edit: this comment is outdated ;)

Thanks, @mhsdesign (also for allowing static titles / uriPathSegments, that are not EEL expressions).

Perhaps we could simplify some parts as we do not need to cover the default behaviour any more. I'll think about it...

@mhsdesign mhsdesign marked this pull request as draft June 1, 2023 09:56
@mhsdesign
Copy link
Contributor Author

patch is released ^^ https://github.com/neos/neos-ui/releases/tag/8.3.1

this makes our hacky DelegatingDocumentTitleNodeCreationHandler obsolete
@mhsdesign mhsdesign force-pushed the bugfix/45-templatingDocumentTitleNodeCreationHandler branch from 323b8ea to 5794564 Compare June 30, 2023 21:29
@mhsdesign mhsdesign marked this pull request as ready for review June 30, 2023 21:29
@mhsdesign mhsdesign changed the title BUGFIX: Position TemplatingDocumentTitleNodeCreationHandler to the end BUGFIX: Position TemplateNodeCreationHandler to the end Jun 30, 2023
@mhsdesign mhsdesign requested a review from Sebobo July 3, 2023 17:57
@Sebobo
Copy link
Member

Sebobo commented Jul 10, 2023

Do we need a min version of the Neos.Ui in our dependencies then?

@mhsdesign
Copy link
Contributor Author

actually yes that would be the cleanest - ill fix that ^^

@mhsdesign
Copy link
Contributor Author

@Sebobo adjusted via e2d9388

@mhsdesign mhsdesign merged commit 4f3df4f into 2.0 Jul 22, 2023
@mhsdesign mhsdesign deleted the bugfix/45-templatingDocumentTitleNodeCreationHandler branch July 22, 2023 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Setting the title via template on a document not working
3 participants