Skip to content

differentiate NewsArticle, TechArticle and BlogPosting#735

Open
lucrousseau21 wants to merge 3 commits into
mainfrom
feat/article-schema
Open

differentiate NewsArticle, TechArticle and BlogPosting#735
lucrousseau21 wants to merge 3 commits into
mainfrom
feat/article-schema

Conversation

@lucrousseau21
Copy link
Copy Markdown
Member

Closes #674

@lucrousseau21 lucrousseau21 changed the title add 'kind' field to blog posts and update related tests and structured data handling differentiate NewsArticle, TechArticle and BlogPosting May 5, 2026
@lacatoire
Copy link
Copy Markdown
Member

Bonne idée d'introduire un champ kind explicite plutôt que de dériver via isTechCategory. Trois points à reprendre avant merge :

  • Régression silencieuse sur ~54 articles tech : sans kind dans le frontmatter, ils tombent en BlogPosting au lieu de TechArticle (Symfony, PHP, Architecture, DevOps, IA…). Soit auto-dériver kind depuis category quand le frontmatter ne le précise pas, soit backfiller les .mdx concernés.
  • src/app/sitemap-news.xml/route.ts:50 : news:image absent alors que les 6 news ont une image — manque d'opportunité.
  • src/__tests__/components/ui/ResponsiveImage.test.tsx:71 : l'assertion sizes a été supprimée plutôt qu'asserter le nouveau comportement de next/image. À restaurer ou documenter.

Un test bout-en-bout sur un article réellement tech (qui passe par getPostBySlugarticleJsonLd) bloquerait toute régression future sur ce mapping.

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.

SEO: différencier NewsArticle et BlogPosting selon le type d'article

2 participants