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

[Bug] Scripts de synchros rdvi / rdvsp #2148

Open
aminedhobb opened this issue Jun 19, 2024 · 3 comments · May be fixed by #2217
Open

[Bug] Scripts de synchros rdvi / rdvsp #2148

aminedhobb opened this issue Jun 19, 2024 · 3 comments · May be fixed by #2217
Assignees
Labels
bug Something isn't working F3 P0 critique

Comments

@aminedhobb
Copy link
Collaborator

aminedhobb commented Jun 19, 2024

Nous avons eu plusieurs problèmes récemment de synchronisations rdvi <-> rdvsp: https://mattermost.incubateur.net/betagouv/pl/kgjuohpzrpb7b8ajgfboxtaupw

Il faudrait faire un script rdvi qui génère un fichier (csv ou json) avec pour chaque usager actif :

  • Son id rdvsp
  • Les ids rdvsp des organisations auxquelles il appartient

Un script rdvsp qui prend en entrée ce fichier et qui:

  • Ajoute les usagers dans les organisations où ils ne sont pas présents
  • Relance un webhook dans pour les organisations rattachées à l'usager et non rattachées dans rdvi (ou alors supprime la liaison sur rdvsp?)
  • Récupère les ids des usagers qui ont été supprimés (pour pouvoir nullifier le rdv_solidarites_user_id sur rdvi)
    => ce script devrait pouvoir être lancé en dry run
@aminedhobb
Copy link
Collaborator Author

aminedhobb commented Jul 22, 2024

Pour info je mets en PJ ce que donnait les résultats du lancement des scripts associés sur rdvsp (fait par @Holist le 19/07/2024 ).

Usagers présents dans une orga rdv-i et pas sur rdvs-p

  • La très grande partie des usagers concerne l'organisation Appel à projets - Decazeville & Villefranche (id rdvsp 4090, rdvi 282).
    Dans le backlog j'ai retrouvé l'issue [Aveyron] Rajouter des users à une organisation #1662 qui consistait à rajouter des usagers à cette organisation. Je me dis qu'il est probable que les usagers aient été ajoutés à l'orga sur rdvi et pas sur rdvsp. @amaurydubot est-ce qu'on les rajoute à l'orga sur rdvsp du coup ?

  • Il existe quand même quelques usagers qui sont présents dans des orgas rdvi et pas rdvsp (notamment une dizaine sur l'ADS de Montargis et sur l'orga CFPPA des Ardennes). Le problème c'est qu'on ne sait pas si l'usager n'a pas été bien créé sur l'orga rdvsp, ou si l'usager a été enlevé sur rdvsp et que ça ne s'est pas répercuté sur rdvi (cette dernière option est la plus probable). Que penses-tu qu'on devrait faire pour ces cas-là @amaurydubot ? Vu les demandes du Loiret je me demande si on ne devrait pas plutôt les ajouter à l'orga sur rdvsp plutôt que de les enlever de l'orga sur rdvi

Usagers présents sur une orga rdvsp et pas sur rdvi

Ces cas sont nombreux mais s'expliquent très facilement: ils concernent majoritairement les organisations qui utilisent rdvsp avant rdv-insertion ou sur d'autres cas d'usages:

  • Lorsqu'un un usager est rapatrié sur rdvi via les webhooks de rdvsp, on ne rapatrie pas toutes les organisations auxquelles il appartient. Du coup s'il appartenait à d'autres orgas elles ne seront pas ajoutées.

  • Si l'orga rdvi a été configurée après la prise de rdv sur l'orga rdvsp, les usagers ne remontent pas dans l'organsation (logique)

  • je remarque par ailleurs qu'on a une organisation qui est injustement taggée comme rdv_insertion (id rdvsp 91 - Centre Médico Social de Saint Vallier)

Même si ces cas ne sont pas étonnant, ils n'en restent pas moins problématiques: on n'affiche pas la même liste d'orga sur rdvsp et rdv-i. Ainsi l'affichage n'est pas cohérent et peut être source de bug (notamment si ajoute l'usager à une orga déjà rattachée depuis rdvi).
=> Il faudrait probablement ajouter l'usager à toutes ces organisations côté rdvi.

Usagers supprimés sur rdvsp et pas sur rdvi

Ces cas sont sans doute dus aux problèmes de synchronisation et d'ordre de process des webhooks.
J'ai enlevé le lien avec l'usager rdvsp correspondant pour tous ces usagers.

organisations_not_in_rdvsp.json
rdvi_rdvsp_coherence.log
organisations_only_in_rdvsp.json

@aminedhobb
Copy link
Collaborator Author

Pour info la PR #2245 est censée remédier aux problèmes des usagers rappatriés depuis RDVSP

@amaurydubot
Copy link
Collaborator

amaurydubot commented Aug 7, 2024

Merci pour ce travail et l'exposé des résultats du lancement du script :

  • S'agissant de l'Aveyron : on peut rajouter tous les usagers concernés de l'organisation Appel à projets - Decazeville & Villefranche (id rdvi 282) à RDV SP oui ;
  • S'agissant des autres cas (comme Loiret) où l'on a l'usager sur rdv-i mais pas sur RDV SP : je pense qu'on peut les ajouter à l'organisation équivalente RDV SP oui (ça permettra notamment à nos utilisateurs sur rdv-insertion de les retirer de telle ou telle organisation s'ils le souhaitent sans avoir le record not found).
  • S'agissant des usagers présents sur une organisation RDV SP mais pas sur rdv-i : du coup si je comprends bien on pourrait rapatrier tous les usagers concernés dans des orgas rdv-insertion a posteriori c'est ça (dans le cas où le rendez-vous de l'usager a été pris sur RDV SP avant la création de l'organisation rdv-insertion) ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working F3 P0 critique
Projects
None yet
4 participants