- First commit to issue fork.
- π©πͺGermany tobiasb Berlin
fyi: The weight should be a integer and show_name a boolean. I fixes this already in the fork.
- Status changed to Needs work
4 months ago 9:46pm 26 August 2024 - πΊπΈUnited States japerry KVUO
With π Missing config schema for field, widget and formatter Fixed committed, this likely needs a reroll. Also, would like to see some tests similar to the other issue before it gets committed.
- π·πΊRussia ilya.no
I've been using patch, made from the MR, via https://git.drupalcode.org/project/social_media_links/-/merge_requests/1..., but after upgrading to D10.3.2 I've got error, as described in https://www.drupal.org/project/social_media_links/issues/3461790 π Schema error after updating to drupal 10.3 Closed: cannot reproduce .
I haven't traced the issue, but it seems to be related to the solution with src/Plugin/Config/Schema/Platforms.php, because when I removed corresponding changes and added code from #12 patch, then the issue was gone.
I'm attaching a bit updated patch with several fixes in order to have ability for deploy.
I think, that issue's priority should be set to Critical, because currently it's not possible to import configuration.
In order to reproduce, need to create page manager variant and add "Social Media Links" block. - π«π·France colin.eininger
I have created new drupal 10.2 and 10.3 instances, installed social_media_links module with the patch from MR!12. Added a block and exported the configuration. Then removed the block and imported the configuration.
Everything works as expected.
In order to troubleshoot your problem @ilya.no, can you describe all minimal step to reproduce ? In the other issue you said "Need to create page manager page". Is that the layout builder your refering to ?
- Assigned to colin.eininger
- π«π·France colin.eininger
Ok, I wasn't on 10.3 during my tests. Tried again and reproducing the error. I'm looking at the issue rn.
- π«π·France colin.eininger
colin.eininger β changed the visibility of the branch 3023172-missing-config-schema to hidden.
- π«π·France colin.eininger
colin.eininger β changed the visibility of the branch 3023172-missing-config-schema to active.
- Issue was unassigned.
- Status changed to Needs review
4 months ago 11:29am 29 August 2024 - π«π·France colin.eininger
I fixed the error on drupal 10.3.
When drupal retrieve a map key, it try to guess which type its parent is and expect the parent to be either a MapDataDefinition or a SequenceDataDefinition.
We were using the ListDataDefinition which is unexpected. I switched our social_media_links.platforms type to use SequenceDataDefinition (which inherit from ListDataDefinition).
I tested the patch on drupal 10.2 and 10.3.