-
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
Support collectivites d'outremer #218
Conversation
…avec codeDepartement='om' et latng basée sur ce qui s'est fait sur l'app android. cf https://github.com/CovidTrackerFr/vitemadose-android/blob/develop/app/src/main/java/com/cvtracker/vmd/data/OutreMerEntry.kt#L3
…in departements.json
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.
Nice! I hope there is no conflict with #211
src/views/vmd-home.view.ts
Outdated
@@ -35,7 +35,15 @@ export class VmdHomeView extends LitElement { | |||
} else { | |||
const commune = event.detail.commune | |||
const departements = await State.current.departementsDisponibles() | |||
const departement = departements.find(({ code_departement }) => code_departement === commune.codeDepartement) | |||
if(!departements.find(d => d.code_departement==='om')) { |
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.
It's not better to add this behavior in the State.current.departementsDisponibles()
method?
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.
Oui, je suis d'accord
Après, ce qui serait encore mieux, c'est qu'on ait pas du tout à le faire :-D (eg, que le back le rajoute :) )
@fcamblor J'ai appliqué les petits changements que j'avais suggéré. Tout est bon pour moi. Si tu n'y vois pas d'inconvénient, on peut la valider :) |
Parfait merci @Luwangel :) |
Fix for #182
om
mis à dispo par le back (cf Pas d'infos centres sur les collectivités d'outremer vitemadose#384)codeDepartement='om'