- Issue created by @wim leers
- Issue was unassigned.
- Status changed to Needs review
10 months ago 1:28pm 22 January 2024 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Note that without ✨ [PP-1] Mark parts of CKEditor 5 and Editor config schema as fully validatable Postponed this will not have any effect due to the way 📌 Configuration schema & required keys Fixed works (see the corresponding change record: https://www.drupal.org/node/3404425 → ).
None of the config schema types in
ckeditor5.schema.yml
are "absolute roots": none of them are oftype: config_object
ortype: config_entity
, and hence none of these are enforced just yet. Only after #3412361 lands will this cause behavior changes.If this issue lands first, we'll need to restore https://git.drupalcode.org/project/drupal/-/merge_requests/6026/diffs?co... there, if the other issue lands first, we'll need to cherry-pick that hunk over here.
The commit order doesn't matter.
- Status changed to RTBC
10 months ago 10:03am 24 January 2024 - 🇧🇪Belgium borisson_ Mechelen, 🇧🇪
I think this is looks good, it's to be expected that nothing changes in the tests yet.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Matching title of 📌 Follow-up for #3364109: opt in already validatgable simple config to FullyValidatable Active .
- Status changed to Fixed
10 months ago 4:02pm 26 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.