Repository navigation
Template pages: "What is…" heading uses the SEO keyword, with a/an by sound - #82
Merged
Merged
Conversation
… sound The definition heading now asks about the template's SEO keyword, falling back to its name. The article follows pronunciation rather than the first letter (an AI…, an online…, a PDF…, a unified…, an hourly…), and acronyms keep their capitals instead of being lowercased. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016GGuBqTRpJoF1ALgzbBRtg
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
 [Low impact] The PR appears safe to merge. SummaryTemplate definition headings now use the SEO keyword, fall back to the template name, and preserve acronym capitals.
Reviews (3) · Last reviewed commit: "Template headings: pick the article from..." · Reviewed by Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
… on "an"
Addresses review: "SaaS" was read letter by letter ("an SaaS"), and the
"uni" exception caught "uninsured"/"unimportant" ("a uninsured").
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016GGuBqTRpJoF1ALgzbBRtg
Collaborator
Author
|
@greptileai please re-review. 10421a5 addresses both findings. Generated by Claude Code |
…ed word "RSVP-based registration form" now gets "an", as it did before the acronym change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016GGuBqTRpJoF1ALgzbBRtg
Collaborator
Author
|
@greptileai please re-review. 0ddc388 fixes the hyphenated-acronym case. Generated by Claude Code |
Collaborator
Author
working as intended
|
vexche
enabled auto-merge (squash)
October 21, 2026 18:19
vexche
disabled auto-merge
October 22, 2026 13:24
This branch was successfully deployed
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
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.
Summary
On template detail pages (
/templates/<slug>), the heading above the definition section changes in three ways:All of it lives in
templateCopy()/article()insrc/lib/template-page.ts. The FAQ heading, CTA and SEO title are unchanged.Linear
No ticket. Ticket creation was skipped for this change.
Testing
src/lib/template-page.test.mjscovering the keyword heading, the name fallback, the sound-based article (including SaaS/SAAS, uninsured/unidentified and hyphenated acronyms) and acronym casing.npm test: 34 pass, 0 fail.tsc --noEmitand ESLint on the changed files are clean.next buildcouldn't complete because the sandbox can't reach the blog's Ghost host. That is unrelated to this change, and Vercel's build is the check here.🤖 Generated with Claude Code
https://claude.ai/code/session_016GGuBqTRpJoF1ALgzbBRtg