[core.entity_view_display.node.article.default:third_party_settings.layout_builder.sections.0.third_party_settings.lb_plus] => missing schema

Created on 23 January 2024, 10 months ago
Updated 19 March 2024, 8 months ago

Problem/Motivation

[core.entity_view_display.node.article.default:third_party_settings.layout_builder.sections.0.third_party_settings.lb_plus] => missing schema

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Fixed

Version

2.1

Component

Code

Created by

heddn Nicaragua

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @heddn
  • heddn Nicaragua
    third_party_settings:
        sections:
            third_party_settings:
              lb_plus:
                uuid: 3084b21e-317d-44b3-9d74-5e3d126bdc0c
    

    This is what is being complained about.

  • Status changed to Postponed: needs info 10 months ago
  • πŸ‡ΊπŸ‡ΈUnited States Tim Bozeman

    Hmm, I feel like you shouldn't need to write schema for third_party_settings πŸ€”

    How can I go about reproducing this error?

  • First commit to issue fork.
  • Merge request !39Add missing third_party schema β†’ (Merged) created by zrpnr
  • Status changed to Needs review 9 months ago
  • πŸ‡ΊπŸ‡ΈUnited States zrpnr UTC-7

    This error occurred for me when adding the `layout_builder_restrictions` module and setting default custom layouts defined in my theme.
    Exporting config creates the following lines in the relevant `entity_view_display` files such as such as `core.entity_view_display.node.article.default.yml`

    Under:
    ```yaml
    third_party_settings:
    layout_builder:
    ```
    In `sections`

    For example:

    ```yaml
    lb_plus:
    uuid:
    ```

    and
    ```yaml
    lb_plus:
    default_section:
    layout_plugin:
    label:
    ```

    Adding definitions for this in `lb_plus.schema.yml` seems to solve the errors, tested both locally and on our CI with the patch from the MR above.

    Credit to @gabesullice for figuring out the syntax for the missing schema.

  • Status changed to Fixed 9 months ago
  • πŸ‡ΊπŸ‡ΈUnited States Tim Bozeman

    LGTM! Thank you very much zrpnr!

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

Production build 0.71.5 2024