- Issue created by @wim leers
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
🐛 PHPUnit Next Major tests failing Active is in, so this is unblocked!
- Merge request !1239Draft: Adopt `AtLeastOneOf` validation constraint for cardinality validation → (Open) created by thoward216
- 🇬🇧United Kingdom thoward216
Have updated the the MR and things are looking better. Now getting errors around the validation for the version constraint that was added/uncommented.
https://git.drupalcode.org/project/experience_builder/-/jobs/5827036#L4725 - 🇬🇧United Kingdom thoward216
I think this is failing as 'versioned_properties' does not contain the hash of the 'active_version' as its simply keyed as 'active'. So I'm not sure this is the correct constraint to be added here?
- 🇬🇧United Kingdom thoward216
Coming back around to this, as per #10.
A new components config with one version looks like this:
uuid: d973b1d2-df01-4f89-a309-fe83f34c9d9a langcode: en status: true dependencies: config: - experience_builder.js_component.test active_version: 8fe3be948e0194e1 versioned_properties: active: settings: prop_field_definitions: { } fallback_metadata: slot_definitions: { } label: test id: js.test provider: null source: js source_local_id: test category: '@todo'
New versions created adds them under `versioned_properties` as expected but as you can see above the hash `8fe3be948e0194e1` is not in `versioned _properties`. So either the above isn't structured as expected (though I'm sure there are a number of tests around this) OR the original constraint that was commented to add later maybe stale?