CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

Ajout d'un sitemap #194

Closed Luwangel closed 3 years ago

Luwangel commented 3 years ago

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

fcamblor commented 3 years ago

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 commented 3 years ago

@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 commented 3 years ago

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

fcamblor commented 3 years ago

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

Plusieurs subtilités :

Luwangel commented 3 years ago

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

fcamblor commented 3 years ago

Let's go :-)

fcamblor commented 3 years ago

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 )