-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Member types: Prepare container support #20715
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
Member types: Prepare container support #20715
Conversation
This reverts commit ad213a2.
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.
Pull Request Overview
This PR enables container (folder) support for Member Types in Umbraco CMS. Previously, the MemberTypeContainerRepository was a no-op implementation that returned empty results, preventing the organization of member types into folders. This change replaces it with a full implementation that inherits from EntityContainerRepository, bringing member types in line with content types, media types, and data types.
- Replaced no-op
MemberTypeContainerRepositorywith functional implementation extendingEntityContainerRepository - Added
MemberTypeContainerto allowed container types inEntityContainerRepository - Registered
MemberTypetoMemberTypeContainermapping inEntityContainermodel
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/Umbraco.Infrastructure/Persistence/Repositories/Implement/MemberTypeContainerRepository.cs | Replaced no-op implementation with functional repository extending EntityContainerRepository |
| src/Umbraco.Infrastructure/Persistence/Repositories/Implement/EntityContainerRepository.cs | Added MemberTypeContainer to the list of allowed container types |
| src/Umbraco.Core/Models/EntityContainer.cs | Added MemberType to MemberTypeContainer mapping in ObjectTypeMap dictionary |
Zeegaan
left a comment
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.
Looks good to me 👍
This includes the minimum necessary to allow Deploy to prepare support for member type containers, expected in 17.1.
The intention is that it will work at a service level, but not be supported yet via the management API or the backoffice.
See #20706 for more details.