Skip to content

Fix canonical base to include /docs mount path#421

Open
Dgiordano33 wants to merge 1 commit into
mainfrom
docs/canonical-docs-path
Open

Fix canonical base to include /docs mount path#421
Dgiordano33 wants to merge 1 commit into
mainfrom
docs/canonical-docs-path

Conversation

@Dgiordano33

Copy link
Copy Markdown
Collaborator

Hotfix for #420. The global seo.metatags.canonical was set to the bare origin https://www.checklyhq.com, so Mintlify emitted canonicals without the /docs mount (e.g. https://www.checklyhq.com/detect/overview instead of .../docs/detect/overview) — non-self-referential on every non-overridden page.

This sets the base to https://www.checklyhq.com/docs so canonicals resolve to the real served path.

Verified on prod after #420:

  • ✅ Per-page canonical: overrides emit verbatim — /docs/cli/overview/https://www.checklyhq.com/docs/cli/overview/ (www + /docs + trailing slash).
  • ❌ Global base without /docs dropped the mount — fixed here.

After deploy, non-overridden pages should read https://www.checklyhq.com/docs/<path> (www + /docs, no trailing slash — a single 308 to the served slash URL).

🤖 Generated with Claude Code

seo.metatags.canonical was set to the bare origin (https://www.checklyhq.com),
which made Mintlify emit canonicals WITHOUT the /docs mount — e.g.
https://www.checklyhq.com/detect/overview instead of .../docs/detect/overview
(the former only resolves via a redirect, so canonicals were non-self-referential).

Include /docs in the base. Verified on prod that Mintlify appends the internal
page path to this base, and that per-page `canonical:` frontmatter overrides are
emitted verbatim (e.g. cli/overview keeps its trailing slash).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant