Repository navigation
Decode HTML entities in Horse Hospital titles - #765
Merged
Merged
Conversation
Squarespace writes the event name into the JSON-LD HTML-escaped, so a title with an ampersand came through as "&". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
cinemas/thehorsehospital.com/transform.js: decode the JSON-LD event name withhtml-entitiesbefore stripping the venue suffix, the same wayalexandrapalace.comdoes.common/tests/test-titles.json: replace the&input that Update title normalization test #764 brought in with the decoded title. Its expected output is unchanged.Why
Squarespace writes the event name into the JSON-LD with HTML escaping, and
JSON.parseleaves that alone. So the listing showed'RAFT' & The Parlour Collective & 'The Moon Over The Alley'. Of the venues in the latest transformed release, it's the only one with an&in a title.Reviewer notes
The snapshot doesn't change because none of the recorded events have an ampersand in their name. The Horse Hospital and normalize-title tests pass, and so does lint.
🤖 Generated with Claude Code