[pending ParakhAI-frontend#444] test: canonical/OG/Twitter meta tags - #14
Draft
saqibmanan wants to merge 1 commit into
Draft
saqibmanan wants to merge 1 commit into
saqibmanan wants to merge 1 commit into
Conversation
…rontend#444) Covers CivicDataLab/ParakhAI-frontend#444 (open, dev, >72h old), which adds canonical link, Open Graph and Twitter Card metadata to the homepage layout. Gated with pending_pr so it skips in CI until that PR merges.
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.
Pending: covers open CivicDataLab/ParakhAI-frontend#444 (head
237a68b). Skipped in CI until it merges. Safe to merge before it.Source PR
CivicDataLab/ParakhAI-frontend#444 — "feat(seo): add canonical link tags and Open Graph meta tags" (open, base
dev, created 2026-07-31, >72h old, no existing coverage).What behaviour changes
app/layout.tsxcurrently exports a bare{ title, description }metadata object. The PR adds:metadataBase+alternates.canonical: "./"→ an absolute<link rel="canonical">on every page.openGraph.{title,description,url,siteName,locale,type}→og:*meta tags.twitter.{card,title,description}→ atwitter:cardmeta tag.title.templateso child routes get%s | ParakhAI(and drops the dashboard layout's now-redundant hardcoded suffix).Category chosen
api(pure HTTP/HTML check viarequests, no browser or login needed — the homepage/is one of the few routes not gated by the auth middleware). No other category fits: this isn't a user flow (e2e) and touches no security/perf-sensitive surface.Red → green → skip proof
1. Collection under CI's real filter (
pytest tests/api/ -m "api"):2. RED — marker stripped, run against
dev.parakh.civicdataspace.in(current/unmerged behaviour):3. GREEN — marker stripped, PR head (
237a68b) served locally (next dev,NEXT_PUBLIC_PLATFORM_URL=https://parakh.civicdatalab.in, stub Keycloak/NextAuth env since the homepage itself needs no session):Confirmed the rendered HTML directly:
<link rel="canonical" href="https://parakh.civicdatalab.in"/>,og:title/og:description/og:url/og:locale/og:type, andtwitter:card=summaryall present.4. Marker restored — re-ran against dev, confirmed the same
SKIPPED ... pending_pr ParakhAI-frontend#444: not merged yetas step 1.Gaps / not covered
/dashboard→.../dashboard) — not checked; the homepage check is the representative case and the dashboard route needs a session.og:image— the PR itself says this is deliberately out of scope (no suitably-sized asset exists yet; tracked as a separate issue on the source PR).readonly: not applicable — this test only ever reads/, but it isn't markedreadonlysince the feature isn't onmain/prod yet. Follow-up once #444 ships to prod.Known edge (per pr-test-sync skill §9)
Merged isn't deployed: if
dev's deploy fails its smoke gate after #444 merges and rolls back, this test will go red against the old page — that's a real signal, not a flake.Generated by Claude Code