-
Notifications
You must be signed in to change notification settings - Fork 56
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 plateforme Clikodoc #530
base: main
Are you sure you want to change the base?
Conversation
Some centers have multiples pratices in the same location. With cache, we avoid having to call multiple times the API with the same address.
J'ai enfin un code fonctionnel, mais j'ai deux soucis :
|
Oubliez ce que j'ai dit, de un je ne sas pas compter et de deux je ne sais pas faire une comparaison, maintenant je n'ai que 3 calls pour récupérer un token et j'ai gagné 15sec par centre avec des créneaux 🚀 😅 |
Après moultes itérations, j'ai fait passé mon pire centre de 1m30 à 12sec, je pense qu'on ne fera pas mieux. Plus qu'à nettoyer le code et implémenter mes tests \o\ |
Hi ✋ Des news sur l'inté de cette plateforme ? :-) |
Je n'ai pas vraiment eu de temps à y consacrer et je ne pense pas que j'en aurais tout de suite. Si quelqu'un veut reprendre le sujet, vous êtes le bienvenue. Il y a effectivement "la révolution des créneaux quotidiens" à gérer mais de la manière dont j'ai codé le bazar les informations sont déjà récupérées et stockées (temporairement), il n'y a qu'à faire remonter les infos, probablement trois quatre lignes de debug à enlever et des test à rajouter (:cold_sweat:) |
** Description **
Ma première ébauche pour l'ajout de la plateforme Clikodoc. Il a encore un certain nombre de choses qui ne fonctionnent pas, du code de debug qui traîne, et des tests inexistants à rajouter. Niveau architecture, c'est un mix de ce qui a été fait sur Doctolib et ordoclic, plus ma sauce maison.
** TODO list **