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

Updates the schema version in theme.json files #169

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

orballo
Copy link
Contributor

@orballo orballo commented Dec 30, 2024

Problem

The linter is warning that the schema version should be 3, and when using the theme/styles internally, WordPress is already transforming the version from 2 to 3 after reading the files from disk.

Solution

Updates the version in all the theme.json files for theme and styles.

@orballo orballo changed the title Updates the schema version in theme.json files [Fix] Updates the schema version in theme.json files Dec 30, 2024
@orballo orballo changed the title [Fix] Updates the schema version in theme.json files Updates the schema version in theme.json files Dec 30, 2024
@amjadr360
Copy link
Contributor

While migrating theme.json from v2 to v3, we also need to adjust properties to align with v3. Here are the required changes. Additional testing will be needed after these changes.

@orballo
Copy link
Contributor Author

orballo commented Jan 3, 2025

I see. The linter didn't flag anything of that :/ I'll have a look at it next week, since this is low priority.

@orballo orballo assigned orballo and amjadr360 and unassigned orballo Jan 3, 2025
@orballo
Copy link
Contributor Author

orballo commented Jan 3, 2025

Amjad is going to look into this instead of me.

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.

2 participants