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

Update shape of theme.json file #3133

Closed
wants to merge 1 commit into from
Closed

Conversation

pbking
Copy link
Contributor

@pbking pbking commented Jan 28, 2021

Changes proposed in this Pull Request:

In accordance with
WordPress/gutenberg#28110

the theme.json file has been updated to reflect the new expected shape.

This was tested locally with Gutenberg 9.8.2 on the /master branch. This will NOT work on .com until that version of Gutenberg has been deployed there. However ONCE IT IS then fixes for #3106 and #3122 will no longer work.

Related issue(s):

#3106 are related as is #3122

WordPress/theme-experiments#182 is similar but for TT1-blocks.

In accordance with
WordPress/gutenberg#28110

the theme.json file has been updated to reflect the new expected shape.
@pbking pbking force-pushed the update/spearhead-shape-of-json branch from 777cc66 to 7268161 Compare January 28, 2021 19:53
@pbking pbking changed the title Update shame of theme.json file Update shape of theme.json file Jan 28, 2021
@scruffian
Copy link
Member

I guess we'll have to time the merging of this one carefully...

@simison
Copy link
Member

simison commented Jan 29, 2021

Are any of the themes on .com using theme.json in production? My hunch is that they're not, so it's ok if there's a short window of breakage after GB upgrade on .com, since it's just for internal testing.

@scruffian
Copy link
Member

Are any of the themes on .com using theme.json in production? My hunch is that they're not, so it's ok if there's a short window of breakage after GB upgrade on .com, since it's just for internal testing.

Yeah, that's right, it's not vital :)

@scruffian
Copy link
Member

What version of Gutenberg is this change shipping in?

@pbking
Copy link
Contributor Author

pbking commented Feb 5, 2021

@scruffian
Copy link
Member

Moving to blocked until 9.9 ships on wpcom

@simison
Copy link
Member

simison commented Feb 15, 2021

FYI @scruffian @pbking 9.9.2 is on .com. p7DVsv-auF-p2

@pbking
Copy link
Contributor Author

pbking commented Mar 25, 2021

this has been corrected elsewhere, closing.

@pbking pbking closed this Mar 25, 2021
@pbking pbking deleted the update/spearhead-shape-of-json branch March 25, 2021 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants