Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(sitemap): Sitemap paginé #152

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

fix(sitemap): Sitemap paginé #152

wants to merge 13 commits into from

Conversation

ggounot
Copy link
Contributor

@ggounot ggounot commented Dec 24, 2024

https://trello.com/c/2MXHfwXs/323-g%C3%A9n%C3%A9ration-de-multiples-fichiers-sitemap-contenant-tous-les-services-et-toutes-les-structures

Documentation Google sur le fichier d'index de sitemaps : https://developers.google.com/search/docs/crawling-indexing/sitemaps/large-sitemaps?hl=fr

Les ViewSet DRF pour les structures et services supportaient déjà une pagination optionnelle. Je l'exploite pour paginer le sitemap.

Le chemin /sitemap.xml indexe les sous-sitemaps des services et structures.

Le chemin /sitemap-services-[page].xml affiche une page de sitemap pour les services.

Le chemin /sitemap-structures-[page].xml affiche une page de sitemap pour les structures.

J'ai enlevé le chemin statique /contribuer qui ne me semble pas utile car navigable naturellement.

J'ai supprimé le test sur l'environnement qui vise, je pense, à ne pas générer de sitemap.xml pour staging. Il me semble inutile car robots.txt empêche déjà l'indexation. Ainsi, on pourra tester le sitemap en staging.

@ggounot ggounot marked this pull request as ready for review December 24, 2024 17:33
@ggounot ggounot requested review from ikarius and jbuget December 24, 2024 23:04
@ggounot ggounot self-assigned this Dec 24, 2024
@ggounot ggounot force-pushed the feature/multiple-sitemap branch from 18d8fba to 72d7298 Compare December 30, 2024 13:35
@ggounot ggounot changed the title Sitemap paginé fix(sitemap): Sitemap paginé Dec 30, 2024
@ggounot ggounot added the bug Something isn't working label Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant