search

CovidTrackerFr / vitemadose-front

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

Ajout Clikodoc #244

Closed hmartinez69 closed 3 years ago

hmartinez69 commented 3 years ago

Cette Pull Request est

Checklist

Description

Cette issue ajoute la plateforme Clikodoc

hmartinez69 commented 3 years ago

@fcamblor New PR ici ;)

hmartinez69 commented 3 years ago

Voici ce que ça donne @Luwangel

Capture d’écran 2021-05-31 à 16 47 40
fcamblor commented 3 years ago

Visible ici : https://dev.vitemado.se/242_clikodoc/

C'est OK pour moi, par contre je ne vois pas encore un seul centre clikodoc dans les centres des JSON département.

Du coup 2 options :

@hmartinez69 je te laisse choisir ce que tu préfères :)

hmartinez69 commented 3 years ago

@Bouska c'est une histoire de combien de temps l'implémentation de Clikodoc en back ? Si vous avez fini dans quelques heures autant la laisser ouverte et on merge dans la foulée

Sinon, je change direct to false le flag promoted et on envoie dans la foulée ;)

fcamblor commented 3 years ago

En vrai, je pense que le plus simple est de merge avec un promoted:false, et je passerai le flag à true en direct sur dev dès que j'aurai le go de @Bouska (l'implem d'une plateforme peut prendre plusieurs jours, voire quelques semaines)

hmartinez69 commented 3 years ago

Visible ici : https://dev.vitemado.se/242_clikodoc/

C'est OK pour moi, par contre je ne vois pas encore un seul centre clikodoc dans les centres des JSON département.

Du coup 2 options :

  • Soit on garde la PR ouverte, et @Bouska nous notifie ici ou sur #241 dès que l'implem back aura été faite (puis on merge dans la foulée)
  • Soit on merge la PR mais sans faire la promotion de la nouvelle plateforme sur la home (@hmartinez69 il faudrait positionner le flag promoted à false en attendant l'implem, et se noter un reminder dès que @Bouska a fini l'implem pour faire la promotion de la plateforme sur la home)

@hmartinez69 je te laisse choisir ce que tu préfères :)

Ducoup je me suis abonné à la PR de @Bouska sur le repo du Back - https://github.com/CovidTrackerFr/vitemadose/pull/530#issue-657808813

Donc je vais switcher le flag promoted on false pour qu'on puisse merge cette PR, et je surveillerai la PR en back, dès que ce sera good, je passerai en true ;)

fcamblor commented 3 years ago

Super, merci pour ta première PR 🎉 ! ;-)

hmartinez69 commented 3 years ago

Merci @fcamblor ! Tu penses que je pourrais rejoindre le mattermost pour contribuer d'avantage ?

fcamblor commented 3 years ago

Je t'ai envoyé ça par mail ;-)