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.

Sign up for GitHub

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

Jump to bottom

Get rid of Neos.NodeTypes package #524

Open
Sebobo opened this issue Aug 11, 2023 · 3 comments · May be fixed by #526
Open

Get rid of Neos.NodeTypes package #524

Sebobo opened this issue Aug 11, 2023 · 3 comments · May be fixed by #526
Assignees
@crydotsnake
Labels
enhancement

Comments

@Sebobo
Copy link
Member

Sebobo commented Aug 11, 2023

We should directly use the specific sub packages we need or migrate to our own components.
@Sebobo
Copy link
Member Author

Sebobo commented Aug 11, 2023

@crydotsnake is this something you would be interested in doing?

@crydotsnake
Copy link
Member

crydotsnake commented Aug 14, 2023 •
edited

@Sebobo I can take a look at it.

@crydotsnake crydotsnake self-assigned this Aug 14, 2023
@crydotsnake crydotsnake linked a pull request Aug 14, 2023 that will close this issue
Draft
@crydotsnake
Copy link
Member

crydotsnake commented Aug 20, 2023 •
edited

List of NodeTypes that needed to be migrate

The list is not complete. But I will update it from time to time.

Content Elements

Old NodeType Name New NodeType Name Done
Neos.NodeTypes:Text Neos.NeosIo:Text ✅
Neos.NodeTypes:Headline Neos.NeosIo:Headline ✅
Neos.NodeTypes:Image Neos.NeosIo:Image ✅
Neos.NodeTypes:TextWithImage Neos.NeosIo:TextWithImage ❌

Mixins

Old NodeType Name New NodeType Name Done
Neos.NodeTypes:TextMixin Neos.NeosIo:Mixins.TextMixin ✅
Neos.NodeTypes:TitleMixin Neos.NeosIo:Mixins.TitleMixin ✅
Neos.NodeTypes:ContentImageMixin Neos.NeosIo:Mixins.ContentImageMixin ✅
Neos.NodeTypes:ImageMixin Neos.NeosIo:Mixins.ImageMixin ✅
Neos.NodeTypes:ImageCaptionMixin Neos.NeosIo:Mixins.ImageCaptionMixin ✅
Neos.NodeTypes:ImageAlignmentMixin Neos.NeosIo:Mixins.ImageAlignmentMixin ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crydotsnake crydotsnake

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FEATURE: remove Neos.NodeTypes package crydotsnake/Neos.NeosIo
2 participants
@Sebobo @crydotsnake