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

Main page title: Show breaking change in VU 2024 iteration #2418

Conversation

EricDunsworth
Copy link
Member

@EricDunsworth EricDunsworth commented Aug 16, 2024

The visual update 2024 iteration's breaking change previously wasn't appearing.

It seems to have been related to structure issues in the JSON-LD file's "breaking" object:

  • Array using curly brackets ( {...} )
  • Key/value pair placed in the array (key was named "style")

This fixes it by:

  • Using square brackets ( [...] ) for the array
  • Replacing the "style" key with a "Style: " prefix in the value

@EricDunsworth
Copy link
Member Author

EricDunsworth commented Aug 16, 2024

Notes:

The visual update 2024 iteration's breaking change previously wasn't appearing.

It seems to have been related to structure issues in the JSON-LD file's "breaking" object:
* Array using curly brackets ( {...} )
* Key/value pair placed in the array (key was named "style")

This fixes it by:
* Using square brackets ( [...] ) for the array
* Replacing the "style" key with a "Style: " prefix in the value
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a content change. Pre-approved upon local testing

@duboisp duboisp added this to the v15.6.0 milestone Aug 19, 2024
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and tested. This does fix and will show the breaking changes that was documented

@duboisp
Copy link
Member

duboisp commented Aug 23, 2024

I confirm this is only a content change

@duboisp duboisp merged commit dd06a45 into wet-boew:master Aug 23, 2024
1 check passed
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