-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Move ReservedFieldNames to type configuration endpoints #16735
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.
Looks good, left a few comments for you to fix 💪
src/Umbraco.Cms.Api.Management/ViewModels/Member/MemberConfigurationResponseModel.cs
Outdated
Show resolved
Hide resolved
src/Umbraco.Cms.Api.Management/ViewModels/Media/MediaConfigurationResponseModel.cs
Outdated
Show resolved
Hide resolved
src/Umbraco.Cms.Api.Management/ViewModels/Document/DocumentConfigurationResponseModel.cs
Outdated
Show resolved
Hide resolved
DataTypesCanBeChanged = _dataTypesSettings.CanBeChanged, | ||
DisableTemplates = _umbracoFeatures.Disabled.DisableTemplates, | ||
UseSegments = _segmentSettings.Enabled, | ||
ReservedFieldNames = _reservedFieldNamesService.GetDocumentReservedFieldNames(), |
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.
Do we want to rename IReservedFieldNamesService.GetDocumentReservedFieldNames()
to GetDocumentTypeReservedFieldNames
, etc.?
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 opted not to, since it seemed like a lot of extra obsolete methods and stuff for not a lot of gain 😄
src/Umbraco.Cms.Api.Management/Controllers/MemberType/ConfigurationMemberTypeController.cs
Outdated
Show resolved
Hide resolved
…ationMemberTypeController.cs Co-authored-by: Elitsa Marinovska <21998037+elit0451@users.noreply.github.com>
Moves the
ReservedFIeldNames
property into configuration endpoints for Document, Media, and Member, since these are required when creating the type, not the entity.Testing
Request the endpoints and ensure they work