-
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 Clikodoc #244
Ajout Clikodoc #244
Conversation
@fcamblor New PR ici ;) |
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.
Merci :) Tout me semble bon. Est-ce que tu as une capture d'écran de la page d'accueil pour voir ce que ça donne ?
Voici ce que ça donne @Luwangel |
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 :) |
@Bouska c'est une histoire de combien de temps l'implémentation de Clikodoc en back ? Sinon, je change direct to false le flag promoted et on envoie dans la foulée ;) |
En vrai, je pense que le plus simple est de merge avec un |
Ducoup je me suis abonné à la PR de @Bouska sur le repo du Back - CovidTrackerFr/vitemadose#530 (comment) 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 ;) |
Super, merci pour ta première PR 🎉 ! ;-) |
Merci @fcamblor ! Tu penses que je pourrais rejoindre le mattermost pour contribuer d'avantage ? |
Je t'ai envoyé ça par mail ;-) |
Cette Pull Request est
Checklist
dev
dev
Description
Cette issue ajoute la plateforme Clikodoc