- Issue created by @mparker17
- Merge request !29Issue #3465601 part 1: Improve configuration schema and how we use it β (Merged) created by mparker17
- Status changed to Needs review
4 months ago 10:11pm 3 August 2024 - π¨π¦Canada mparker17 UTC-4
Okay, Iβve decided to create a merge request for the βsafestβ change, i.e.: the one that only makes changes that maintain compatibility with Drupal 9.0, i.e.: changing certain types from string to path.
- Status changed to Active
4 months ago 11:54am 6 August 2024 - π¨π¦Canada mparker17 UTC-4
Merging the string β path changes.
Moving back to "Active" for the rest of the proposed changes
-
mparker17 β
committed cbc69f18 on 8.x-2.x
Issue #3465601 part 1: Improve configuration schema and how we use it
-
mparker17 β
committed cbc69f18 on 8.x-2.x
- π¨π¦Canada mparker17 UTC-4
Note that part 1 of this change was released in sitemap-8.x-2.0-beta7 β ; but there are still outstanding parts of this issue to be released in the future.
- πΊπΈUnited States dcam
From my own comment in #3465577-18: Split sitemap_book config schema from sitemap schema β :
I'm guessing that a lot of the
nullable
keys were the result of all plugin settings being lumped into a single array. You had to do that so one plugin could leave all the other settings as empty.While this config improvement is going on, any unnecessary
nullable
properties should be removed.