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

Mise à jour conditions éligibilité vaccin #251

Merged
merged 4 commits into from
Jul 17, 2021

Conversation

a2br
Copy link
Contributor

@a2br a2br commented Jun 21, 2021

Cette Pull Request est

  • Un correctif
  • Une nouvelle fonctionnalité

Checklist

  • Si cette PR résoud une issue, j'indique son numéro dans les description
  • Cette PR vise la branche dev
  • Elle n'est pas en conflit avec la branche dev

Description

Mise à jour des conditions d'éligibilité au vaccin

image

a2br added 2 commits June 21, 2021 08:48
Ceci n'est qu'une recommendation, pas une obligation. Sinon, reformuler différemment. Source: https://www.service-public.fr/particuliers/actualites/A14722
- Il faut avoir + de 12 ans
- Suppression du reste, qui était redondant face au 1er point
Copy link
Collaborator

@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 ! J'ai juste une remarque sur la vaccination des adolescents.

index.html Outdated Show resolved Hide resolved
- ajout de la balise sémantique time pour l'accessibilité
- le bloc est maintenant plus compact
- correction d'une erreur dans l'URL affichée
@a2br
Copy link
Contributor Author

a2br commented Jun 21, 2021

Merci pour ton aide! Je viens de rajouter des changements. C'est dans la description de a5a28a8.

image

Je n'ai pas vraiment d'avis tranché sur l'orthographe de ce mot, je corrige juste l'incohérence : partout ailleurs on met un tiret. (pas vraiment en lien avec la PR, mais j'en profite)
@a2br a2br requested a review from Luwangel June 21, 2021 13:40
@Luwangel Luwangel requested a review from fcamblor July 13, 2021 09:29
@Luwangel Luwangel mentioned this pull request Jul 13, 2021
2 tasks
@fcamblor fcamblor changed the base branch from dev to main July 17, 2021 09:12
@fcamblor
Copy link
Collaborator

LGTM, j'ai changé la target vers main pour MEP en suivant :-)

@fcamblor fcamblor merged commit 7291243 into CovidTrackerFr:main Jul 17, 2021
@fcamblor
Copy link
Collaborator

Merci pour la contrib @a2br :)

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.

3 participants