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

#18 Replace Centres by Lieux to have a more generic semantic #22

Merged
merged 7 commits into from
Apr 10, 2021

Conversation

nhumblot
Copy link
Collaborator

@nhumblot nhumblot commented Apr 10, 2021

Fix #18

Transforme les mentions de 'centres' en 'lieux'.

Quelques exceptions sont à noter où la terminologie 'centre' a été conservée:

  • Le mot-clé "Centres de Vaccination Covid19 en France" a été conservé dans les metadonnées pour aider le référencement ("Lieux de Vaccination Covid19 en France" a été ajouté) ;
  • En réponse à la question "Où se faire vacciner" car le changement semblait inadapté => Si on est éligible, on peut être vacciné chez son médecin généraliste, dans un centre de vaccination, à la pharmacie, ou dans tous les établissements de santé et médico-sociaux.
  • Le TypeLieu "vaccination-center" qui conserve le libellé spécifique "Centre de vaccination".

La base de code étant majoritairement en français, ce choix initial a été conservé.

Une attention a été apportée dans State.ts afin de continuer à utiliser les bons champs renvoyés par l'api ${VMD_BASE_URL}/${codeDepartement}.json, à savoir centres_disponibles et centres_indisponibles.

Tout commentaire est le bienvenu.

Il s'agit de ma première contribution sur ce projet, merci d'y apporter tout le soin nécessaire.

index.html Outdated
<title>Le moteur de recherche d’un créneau de vaccination contre la Covid-19 dans un lieu de vaccination en France.</title>
<meta name="title" content="Le moteur de recherche d’un créneau de vaccination contre la Covid-19 dans un lieu de vaccination en France.">
<meta name="description" content="Le site qui répertorie la majorité des lieux de vaccination Covid-19 en France et vous permet de trouver un créneau de vaccination Pfizer-BioNTech, Moderna, ou AstraZeneca dans un lieu de vaccination proche de chez vous.">
<meta name="keywords" content="Vaccination covid-19, Centres de Vaccination Covid19 en France, Lieux de Vaccination Covid19 en France, Covid-19, covid, covid-19, coronavirus, Moderna, Pfizer, AstraZeneca, vaccin, vaccination, campagne vaccinale, comorbidités">
Copy link
Collaborator

Choose a reason for hiding this comment

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

@rozierguillaume tu es OK là-dessus ?

@fcamblor fcamblor changed the base branch from main to dev April 10, 2021 16:05
@fcamblor fcamblor force-pushed the 18-remplacer-centres-par-lieux branch from 8b9349e to 235fa5e Compare April 10, 2021 16:20
@fcamblor fcamblor force-pushed the 18-remplacer-centres-par-lieux branch from 235fa5e to a6240f6 Compare April 10, 2021 16:22
@fcamblor
Copy link
Collaborator

Hello @nhumblot,

Pourrais-tu me sortir de cette PR tout ce qui concerne le contenu stp ?

Il semblerait que le SEO sur "centre de vaccination" soit bien meilleur que "lieu de vaccination", et les membres de l'équipe ont peur que cela impacte négativement notre référencement.
Screenshot_2021-04-10 Google Trends (1)

Si tu peux me faire une second PR à part avec ce reliquat de renommage (afin que ça ne soit pas perdu, au cas où on revienne plus tard sur cette décision), c'est encore mieux (mais aucune obligation si tu galères à la faire... car il y a de bonnes chances pour que cette seconde PR ne soit jamais mergée)

@nhumblot
Copy link
Collaborator Author

nhumblot commented Apr 10, 2021

Ok, je corrige 👍

@nhumblot
Copy link
Collaborator Author

nhumblot commented Apr 10, 2021

@fcamblor Le retour arrière sur le contenu a été effectué et j'ai corrigé le conflit. Je te laisse refaire une passe si tout est ok pour toi. J'ai pris soin de faire le retour arrière sur l'url du routeur également. J'ai fait quelques tests de mon côté et ça me semble bon.

J'ai créé une branche 18-conserver-renommage-contenu qui revert mes deux derniers commits. Une fois que tout sera contrôlé sur celle-ci, je prendrai le temps d'ouvrir une nouvelle PR pour garder le changement sur le repo principal tout soumettant un seul commit nettoyé.

@nhumblot nhumblot requested a review from fcamblor April 10, 2021 17:35
@fcamblor fcamblor merged commit a22d438 into CovidTrackerFr:dev Apr 10, 2021
@fcamblor
Copy link
Collaborator

Merci @nhumblot pour la rapidité c'est parfait 👌

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.

None yet

2 participants