There's a small issue with 6.0-beta9 (and dev) release resulting in WSOD:

Drupal\sdc\Exception\InvalidComponentException: [configuration.label_display] Boolean value found, but a string is required in Drupal\sdc\Component\ComponentValidator->validateProps() (line 205 of /app/web/core/modules/sdc/src/Component/ComponentValidator.php).

It can be easily fixed by changing templates/block/block.html.twig file to something like

{% include 'radix:block' with {
  configuration: {
    label_display: 'above'
  }
} %}

It looks like there's not documentation regarding the expected string values, though.

CommentFileSizeAuthor
#8 3419776-08.patch555 bytesfranknoel

Issue fork radix-3419776

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

milos.kroulik created an issue. See original summary.

milos.kroulik opened merge request !78

milos.kroulik’s picture

milos.kroulik commented
Status: Active » Needs review

I've created a MR with the fix described above, the only difference is that label_display is set to expected value visible.

doxigo’s picture

doxigo commented

The error suggests a fix in the component.yml file, I will check on this later

doxigo’s picture

doxigo commented
Status: Needs review » Fixed

This is fixed Milos, thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jensbns’s picture

jensbns commented

The merge request always sets the label_display to 'visible' and makes you lose all the other objects set in configuration.
I think a better way is to edit the block component block.component.yml to allow types booleans in configuraction.label_displays.

    configuration:
      type: object
      title: Configuration
      description: A list of the block's configuration values.
      properties:
        label_display:
          type: ['string', 'array', 'integer', 'boolean']
          title: Label Display
          description: The display settings for the label.
franknoel’s picture

franknoel commented
Version: 6.0.x-dev » 6.0.0-rc5
StatusFileSize
new 3419776-08.patch555 bytes

I agree with @jensbns, I made a patch containing the mentioned changes for v6.0.0-rc5.

doxigo’s picture

doxigo commented

Thanks guys, updated on the latest dev

danchadwick closed merge request !78