Repository navigation
fix(web): give blog posts and docs pages canonical URLs - #3421
Open
patrickballard wants to merge 1 commit into
Open
patrickballard wants to merge 1 commit into
patrickballard wants to merge 1 commit into
Conversation
Every marketing page declares a canonical URL, but blog posts, docs pages and /deactivate-license didn't. Add them, and point the docs home, which answers at both /docs and /docs/introduction, at /docs.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
This branch was previously 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.
All of cap.so's marketing pages declare a canonical URL, but the 12 blog posts, the 23 docs pages and
/deactivate-licensedon't (checked against the live site). This adds one to each, in the same absolutehttps://cap.so/...form the other pages use:https://cap.so/blog/<slug>https://cap.so/docs/<slug>, except the docs home, which answers at both/docsand/docs/introduction: both point athttps://cap.so/docs, the URL the header, footer and nav link to. Itsog:urlnow says/docstoo./deactivate-license:https://cap.so/deactivate-licenseTests
canonical-urls.test.ts:/deactivate-licenseadded to the list, and two checks in the style of the existing[conversionPath]one for the blog and docs routes. All three fail without the change; the file's other 27 checks still pass. (This file isn't in CI's web test list today.)generateMetadatareturns these canonicals for every blog post and for/docs,/docs/introductionand/docs/recording/studio-mode.bun run typecheck: no errors. Biome clean.

Confidence Score: 5/5
The PR appears safe to merge.Summary
Adds canonical URLs to blog posts, docs pages, and
/deactivate-license.Reviews (1) · Last reviewed commit: "fix(web): give blog posts and docs pages..." · Reviewed by Greptile