feat(blog): external canonical for syndicated posts #7879
+7
−1
Conversation
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for external canonical URLs on syndicated blog posts, following SEO best practices.
- Adds a canonical field to the Mikeal announcement markdown file with an external URL.
- Updates the dynamic routing logic to use the canonical URL from page data if provided.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| apps/site/pages/en/blog/announcements/mikeal.md | Adds a canonical field in front matter for syndicated content. |
| apps/site/next.dynamic.mjs | Adjusts the canonical URL logic to use the provided value when available. |
Comments suppressed due to low confidence (1)
apps/site/pages/en/blog/announcements/mikeal.md:7
- [nitpick] Consider adding an inline comment in the front matter indicating that this canonical URL represents the original source of the syndicated content for SEO clarity.
canonical: https://openjsf.org/blog/in-memory-of-mikeal-rogers
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7879 +/- ##
=======================================
Coverage 75.46% 75.46%
=======================================
Files 101 101
Lines 8311 8311
Branches 218 218
=======================================
Hits 6272 6272
Misses 2037 2037
Partials 2 2 ☔ View full report in Codecov by Sentry. |
avivkeller
approved these changes
Jun 20, 2025
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Matt Cowley <me@mattcowley.co.uk>
|
I’m okay fast tracking this, but I think members of the foundation want to wait until next week. |
AugustinMauroy
approved these changes
Jun 20, 2025
bjohansebas
approved these changes
Jun 20, 2025
flakey5
approved these changes
Jun 21, 2025
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.
Description
To follow SEO best practices, when a blog post is syndicated to our blog from elsewhere, we should set the meta canonical to point to the original post, so that indexing crawlers know this is a syndicated version of the original, rather than competing (near-)identical content.
Validation
On the blog post for Mikeal, the canonical is set to the OpenJSF blog post. On all other blog posts, and all other pages, the canonicals remain as in production.
Related Issues
N/A
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.