Skip to content

add sitemap images and news routes with tests for image metadata#736

Open
lucrousseau21 wants to merge 3 commits into
mainfrom
feat/seo-sitemap-images
Open

add sitemap images and news routes with tests for image metadata#736
lucrousseau21 wants to merge 3 commits into
mainfrom
feat/seo-sitemap-images

Conversation

@lucrousseau21
Copy link
Copy Markdown
Member

Closes #675

@lucrousseau21 lucrousseau21 marked this pull request as ready for review May 6, 2026 13:37
@lacatoire
Copy link
Copy Markdown
Member

Le sitemap images apporte une vraie richesse vs le sitemap principal (title/caption/geo_location). Quelques points avant merge :

  • sitemap-news.xml n'est référencé nulle part : ni dans robots.ts, ni dans un sitemap index. Soit on l'expose dans src/app/robots.ts:21, soit on supprime la route.
  • Spec Google News non respectée : src/app/sitemap-news.xml/route.ts:18 émet tous les posts. La spec n'accepte que les articles < 48 h. À filtrer, et ${post.date} brut sans heure/timezone n'est pas suffisant non plus.
  • src/app/sitemap-images.xml/route.ts:69 : "Illustration d'eco-conception et de sobriete numerique" → accents manquants.
  • La migration <img>next/image dans ResponsiveImage.tsx semble hors scope d'un PR « sitemap » : avec output: "export" + images.unoptimized: true, le bénéfice est nul. À sortir dans un PR dédié.

À tester avant merge : vérifier que output: "export" génère bien out/sitemap-images.xml et pas un index.html dans un dossier.

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: sitemap images dédié pour Google Images

2 participants