LottePitcher / umbraco-admin-only-property

An Umbraco package for properties restricted by user groups
https://marketplace.umbraco.com/package/our.umbraco.community.adminonlyproperty
MIT License
10 stars 8 forks source link

Can create a new data type without specifying the 'underlying' data type #8

Open LottePitcher opened 1 year ago

LottePitcher commented 1 year ago

You can save a data type using the Admin Only Property property editor without having specified the underlying data type, which of course then leads to unexpected behaviour. Would be better if you couldn't save whilst this property is blank

image

leekelleher commented 1 year ago

There's meant to be something with an IValueValidator for the Data Type's configuration field. https://our.umbraco.com/apidocs/v8/csharp/api/Umbraco.Core.PropertyEditors.IValueValidator.html

But I've never fully got my head around how it works... I haven't even implemented these on Contentment! 😳