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

Ajout d'un sitemap #194

Merged
merged 15 commits into from
May 17, 2021
Merged

Ajout d'un sitemap #194

merged 15 commits into from
May 17, 2021

Conversation

Luwangel
Copy link
Collaborator

Premier jet d'implémentation d'un sitemap. Pour le moment je l'ai créé à la main, à voir pour l'optimiser. En l'état, il permet aux moteurs de recherche de connaître les urls canoniques du site.

Il manque les pages des résultats de recherche mais je ne sais pas sous quelle forme les présenter.

J'ai ajouté des slots pour les chronodoses et les lieux de vaccination sur la page brute index.html, dans le but que leur contenu soit référencé par Google. Ils ne s'affichent cependant plus lorsque le framework lit-element prend la main.

J'attends vos retours :)

Todo

  • Create a slot for the chronodose content
  • Lint and simplify the index.html file (move spacing blocks in the page where they are needed)
  • Lint the Router file
  • Create a first sitemap.xml and reference it in the robots.txt file

@fcamblor
Copy link
Collaborator

fcamblor commented May 11, 2021

Je pense qu'on pourrait générer toutes les URLs de recherche possibles du sitemap.xml à partir du https://github.com/CovidTrackerFr/vitemadose-front/tree/dev/tools (en même temps qu'on génère tous les fichiers auto-complete, on pourrait générer toutes les déclinaisons d'URL par commune/département en fonction des tris possibles)

@Luwangel
Copy link
Collaborator Author

@fcamblor Très bonne idée, ça fait sens. Par contre, je pourrai pas le faire ce week-end. Si ça te va, je le ferai dans une autre PR la semaine pro.

@fcamblor
Copy link
Collaborator

Je vais m'en occuper, je pense pas que ça me prenne longtemps :-)

@fcamblor
Copy link
Collaborator

Ça aura pris un peu plus de temps que prévu, mais c'est bon :-)

Plusieurs subtilités :

  • Étant donné qu'on a 35k communes, et que chaque commune possède 2 URLs, on est partis pour générer 70k+ URLs dans le sitemap
  • Google interdit d'avoir plus de 50k URLs par fichier sitemap (source: https://developers.google.com/search/docs/advanced/sitemaps/build-sitemap#general-guidelines)
  • C'est pourquoi j'ai décidé de créer un sitemap index en guise de point d'entrée, et 1 fichier sitemap par département (dans lequel je mets toutes les URLs de tous les communes du département)
  • Tous ces fichiers sont stockés dans public/sitemaps/
  • On a un sitemap dédié aux pages "statiques" (non dynamiques) : public/sitemaps/stemap-pages.xml
  • J'en ai profité pour mutualiser les "définitions" d'urls dans des fichiers partagés entre la partie tooling et la partie sources (typescript) de vmd, afin de bien penser à mettre à jour la manière de générer les URLs si on devait modifier les patterns d'url pour les résultats
  • Une fois la mutualisation de code mise en place, j'ai été un cran plus loin en en profitant pour mutualiser la fonction toFullTextSearchableString() que j'avais dupliqué entre les sources et le tooling ... c'est dorénavant mutualisé !

Copy link
Collaborator Author

@Luwangel Luwangel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super boulot @fcamblor. J'ai fait un peu le tour des changements et ça me paraît top. Il y a juste un conflit à corriger je crois. (Par contre j'étais pas sur l'ordi, #revuedecodesurtelephone, donc j'ai pu passer à côté de certains points :D).

@Luwangel
Copy link
Collaborator Author

Luwangel commented May 16, 2021

@fcamblor J'ai géré le conflit sur cette PR et rebase sur dev. Tout me semble bon maintenant :)

@fcamblor
Copy link
Collaborator

Let's go :-)

@Luwangel Luwangel deleted the seo-sitemap branch May 17, 2021 19:32
@fcamblor
Copy link
Collaborator

Oh, je n'avais pas fait attention, mais j'ai l'impression qu'une régression s'est glissée lors de ton rebase car on a du code de Autocomplete.ts qui n'est plus "mutualisé" entre la partie tooling et la partie Autocomplete (méthode normalize(), sur laquelle un trim() a été appliqué lors de #217 )

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.

2 participants