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

Ajout Clikodoc #244

Merged
merged 3 commits into from
May 31, 2021
Merged

Ajout Clikodoc #244

merged 3 commits into from
May 31, 2021

Conversation

hmartinez69
Copy link
Contributor

Cette Pull Request est

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

Cette issue ajoute la plateforme Clikodoc

@hmartinez69
Copy link
Contributor Author

@fcamblor New PR ici ;)

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.

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 ?

@hmartinez69
Copy link
Contributor Author

Voici ce que ça donne @Luwangel
Capture d’écran 2021-05-31 à 16 47 40

@fcamblor
Copy link
Collaborator

fcamblor commented May 31, 2021

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 Clikodoc: nouvelle plateforme #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 :)

@hmartinez69
Copy link
Contributor Author

@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
Copy link
Collaborator

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
Copy link
Contributor Author

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 Clikodoc: nouvelle plateforme #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 - 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 ;)

@fcamblor fcamblor merged commit 31fc58b into CovidTrackerFr:dev May 31, 2021
@fcamblor
Copy link
Collaborator

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

@hmartinez69
Copy link
Contributor Author

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

@fcamblor
Copy link
Collaborator

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

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