We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Hi there!
It looks like field setting attributes aren't being passed to the field whenever it is a single field collection.
If we have the below example...
[PageType(Title = "Standard page")] public class StandardPage : Page<StandardPage> { [Region] [StringFieldSettings(DefaultValue = "Hello World!")] public List<StringField> Fields { get; set; } [Region] public TestRegion TestRegion { get; set; } } public class TestRegion { [Field] public StringField RegularStringField { get; set; } [Field] [StringFieldSettings(DefaultValue = "Hello World!")] public StringField FancyStringField { get; set; } }
The FancyStringField works as expected, with the settings being pushed through to the field.
FancyStringField
However, with the Fields region, the settings aren't being pushed through.
Fields
It looks like the settings needs to be added to the default ContentTypeField here and then pulled through here.
ContentTypeField
Will try to get a PR together soon, but let me know if I'm missing anything here, thanks :)
The text was updated successfully, but these errors were encountered:
I simply just don’t think this case is covered by the settings. I’ll tag this as a feature request!
Sorry, something went wrong.
PiranhaCMS#2053 Pass fieldsettings to region field collection
3243da4
PiranhaCMS#2053 removed unnecessay assignment to settings in ContentT…
ff5531a
…ypeService
#2053 Pass fieldsettings to region field collection
7efca43
#2053 removed unnecessay assignment to settings in ContentTypeService
7cabc82
Fixed as of #2054
sit-harry
No branches or pull requests
Hi there!
It looks like field setting attributes aren't being passed to the field whenever it is a single field collection.
If we have the below example...
The
FancyStringField
works as expected, with the settings being pushed through to the field.However, with the
Fields
region, the settings aren't being pushed through.It looks like the settings needs to be added to the default
ContentTypeField
here and then pulled through here.Will try to get a PR together soon, but let me know if I'm missing anything here, thanks :)
The text was updated successfully, but these errors were encountered: