Remove em dashes from the manifest and meta description - #14
Merged
vpetersson-bot merged 1 commit intoAug 5, 2026
Merged
Conversation
The app-store fetches this manifest at build time and renders its description and summary straight onto signage-apps.com, so em dashes here show up as em dashes on the store pages (/, /examples/, /timer/). Rewrites the two manifest strings and the page meta description without them. Copy only; no schema, settings or behaviour change.
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.
Why
signage-apps.com (
Screenly-Labs/app-store) fetches this app's.well-known/signage-app.jsonat build time and renders itsdescriptionandsummarydirectly onto the store pages. So em dashes in this manifest become em dashes on/,/examples/and/timer/over there, which is against the house copy style.This was the one remaining source of em dashes on the store after Screenly-Labs/app-store#53; it could not be fixed from that repo because the manifest is fetched from this origin.
What changed
Copy only, three strings:
description:…once it passes — so the same screen…becomes…once it passes, so the same screen…summary:Count down to — or up from — any moment.becomesCount down to or up from any moment.index.htmlmeta description: em dash becomes a colonNo schema, settings, launch template or behaviour change.
Verification
bun run typecheck,bun run lint,bun test(28 pass) andbun run buildall clean. Builtdist/.well-known/signage-app.jsonis valid JSON and both it anddist/index.htmlcontain zero em dashes.Note the store will keep serving the old text until this is deployed and the app-store rebuilds.