Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Block Library: Fix incorrect attributes definitions #36140

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

gziolo
Copy link
Member

@gziolo gziolo commented Nov 2, 2021

Description

Fixes the following schema violation for block.json files reported in #35902:

  • Column block: templateLock - type missing
  • Cover block: templateLock - type missing
  • Group block: templateLock - type missing

@gziolo gziolo requested a review from ajitbohra as a code owner November 2, 2021 11:47
@gziolo gziolo self-assigned this Nov 2, 2021
@gziolo gziolo added [Package] Block library /packages/block-library [Type] Bug An existing feature does not function as intended labels Nov 2, 2021
@gziolo gziolo merged commit c5e695d into trunk Nov 2, 2021
@gziolo gziolo deleted the update/block-json-attributes branch November 2, 2021 12:55
@github-actions github-actions bot added this to the Gutenberg 11.9 milestone Nov 2, 2021
walbo pushed a commit that referenced this pull request Nov 5, 2021
The `templateLock` attribute is mixed between boolean and string. Setting the type as string introduces an regression that will remove `false` on save.

The original PR (#36140) tried to fix a validation issue reported in #35902

The schema error is incorrect. The attribute definition is required to have either a `type` or an `enum`.
andrewserong pushed a commit that referenced this pull request Nov 10, 2021
* Revert "Block Library: Fix incorrect attributes definitions #36140"

The `templateLock` attribute is mixed between boolean and string. Setting the type as string introduces an regression that will remove `false` on save.

The original PR (#36140) tried to fix a validation issue reported in #35902

The schema error is incorrect. The attribute definition is required to have either a `type` or an `enum`.

* Spesify type as string or boolean

* Add unit test
noisysocks pushed a commit that referenced this pull request Nov 10, 2021
* Revert "Block Library: Fix incorrect attributes definitions #36140"

The `templateLock` attribute is mixed between boolean and string. Setting the type as string introduces an regression that will remove `false` on save.

The original PR (#36140) tried to fix a validation issue reported in #35902

The schema error is incorrect. The attribute definition is required to have either a `type` or an `enum`.

* Spesify type as string or boolean

* Add unit test
andrewserong pushed a commit that referenced this pull request Nov 10, 2021
* Revert "Block Library: Fix incorrect attributes definitions #36140"

The `templateLock` attribute is mixed between boolean and string. Setting the type as string introduces an regression that will remove `false` on save.

The original PR (#36140) tried to fix a validation issue reported in #35902

The schema error is incorrect. The attribute definition is required to have either a `type` or an `enum`.

* Spesify type as string or boolean

* Add unit test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Block library /packages/block-library [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants