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

The frontend process now validates the theme configuration based on the JSON schema. #880

Merged
merged 4 commits into from
Dec 19, 2019

Conversation

devbucket
Copy link
Contributor

Description

The new JSON schema for the theme configuration is now validated by the frontend process, to make sure that errors in the configuration don't break the application. The frontend process will fail, if the configuration is invalid.

Type of change

Please add an "x" into the option that is relevant:

  • Bug Fix 🐛 (non-breaking change which fixes an issue)
  • Enhancement 🚀 (non-breaking change which adds functionality)
  • Breaking Change 💥 (fix or feature that would cause existing functionality to not work as expected)
  • Polish 💅 (Just some cleanups)
  • Internal 🏠 Only relates to internal processes.

@devbucket devbucket added the enhancement New feature or request label Dec 18, 2019
@devbucket devbucket self-assigned this Dec 18, 2019
@devbucket devbucket changed the base branch from v6.10.0 to v6.11.0 December 18, 2019 10:10
@alexbridge alexbridge merged commit 85374c8 into v6.11.0 Dec 19, 2019
@alexbridge alexbridge deleted the PWA-2355-validate-json-schema-in-webpack branch December 19, 2019 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants