-
Notifications
You must be signed in to change notification settings - Fork 51
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 de la fonctionnalité permettant de choisir le type de vaccin #143
base: dev
Are you sure you want to change the base?
Changes from 16 commits
1198727
c08569a
bde62ec
7bb403a
053631e
1f3997a
60c6051
2dd2e8c
f22280c
4e84f80
e530d61
669e3f1
16e6076
d692a01
ebe7ada
57f5b02
fe2e3de
6609d34
dcf84f1
50527d7
1c4441a
55dadef
77ad7b0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,10 +16,27 @@ export type TriCentre = { | |
libelle: string; | ||
}; | ||
export const TRIS_CENTRE: Map<CodeTriCentre, TriCentre> = new Map([ | ||
['distance', { codeTriCentre: 'distance', libelle: "Au plus proche" }], | ||
['date', { codeTriCentre: 'date', libelle: "Disponible au plus vite" }], | ||
['distance', { codeTriCentre: 'distance', libelle: "Au plus proche", title:"Les lieux de vaccination les plus proches" }], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Pourquoi cette modification de libellés ? Ça me paraît hors sujet de la PR en cours. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ce n'est pas une modification de libellés, c'est juste que comme j'ai mis des title sur l'autre sélecteur, j'ai ais mis sur celui ci aussi. J'enlève si cela ne vous va pas. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. oh oui my bad je n'avais pas les yeux en face des trous, je croyais que le |
||
['date', { codeTriCentre: 'date', libelle: "Disponible au plus vite", title:"Les créneaux au plus vite" }], | ||
]); | ||
|
||
export type CodeTypeVaccin = 'tous' | 'arnm' | 'adenovirus'; | ||
export const FILTRE_TYPE_VACCIN: Map<CodeTypeVaccin, {codeTypeVaccin: CodeTypeVaccin, libelle: string, title: string}> = new Map([ | ||
['tous', {codeTypeVaccin: 'tous', libelle: "Tous", title: "Tous les vaccins" }], | ||
['arnm', {codeTypeVaccin: 'arnm', libelle: "Pfizer-BioNTech/Moderna", title: "ARN messager" }], | ||
['adenovirus', {codeTypeVaccin: 'adenovirus', libelle:"AstraZeneca/Janssen", title: "à base d'adénovirus" }], | ||
]); | ||
|
||
|
||
export type TypeVaccin = 'AstraZeneca'|'Pfizer-BioNTech'|'Moderna'|'Johnson'|'Janssen'; | ||
export const TYPES_VACCIN: Record<TypeVaccin, CodeTypeVaccin> = { | ||
"AstraZeneca": 'adenovirus', | ||
"Pfizer-BioNTech": 'arnm', | ||
"Moderna": 'arnm', | ||
"Johnson": 'adenovirus', | ||
"Janssen": 'adenovirus', | ||
}; | ||
|
||
const USE_RAW_GITHUB = false | ||
const VMD_BASE_URL = USE_RAW_GITHUB | ||
? "https://raw.githubusercontent.com/CovidTrackerFr/vitemadose/data-auto/data/output" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faut rester backward compatible sur les URLs (si Google a référencé l'une de nos "vieilles" urls, il faut qu'on sache continuer de la résoudre correctement)
Ici, la plupart des changements sur les URLs sont backward incompatibles.
Et dans tous les cas, je m'interroge sur la pertinence de passer tout ce qui se trouve dans les "paramètres avancés" sous forme de Path parameters ... j'en ferais bien des query parameters plutôt que des path parameters (ça résoud le problème de backward incompat par la même occasion)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je ne voulais pas forcément le mettre en paramètre URL le type, c'est juste que je suis parti sur une reprise telle quelle des fonctionnalité comme pour l'autre sélecteur. Mais si on ne veut pas de modification d'URL, on peut revenir dessus.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fcamblor mais passer en path plutot que paramètres c'est mieux pour la SEO ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'un point de vue SEO, c'est mieux de faire passer le typeVaccin en query parameters (comme sur un site ecommerce quand on filtre sur une page liste de produits)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faut trouver un juste milieu entre tout mettre en path parameters (et se tirer une balle pour la backward compat à chaque fois qu'on rajoute un critère de tri/recherche) ou ne mettre que ce que permet à Google de "catégoriser" le site.
Je serais d'avis de mettre :
@francoisBouchet niveau SEO, tu serait ok avec ça ?