gip-inclusion / rdv-insertion

Application permettant de fluidifier le parcours d’accompagnement social et professionnel sur les territoires
https://www.rdv-insertion.fr
13 stars 4 forks source link

[Bug] Statut du rendez-vous qui remonte incorrectement dans l'export rdv- #1528

Closed amaurydubot closed 11 months ago

amaurydubot commented 1 year ago

Comportement observé

L'usager rdv-i 54229 a eu une absence non excusé à un rendez-vous. Ce statut est bien renseigné, tant dans la fiche rdv-insertion que dans la fiche rdv-solidarités.

Toutefois, lors de l'export .csv rdv-insertion, son statut remonte en "Rendez-vous honoré" et ce alors même qu'il n'est pas concerné par un autre rendez-vous par ailleurs (c'est bien de ce dernier rendez-vous non honoré qu'on parle dans l'export).

Comportement souhaité

Rétablir la bonne remontée du statut dans l'export .csv

Michaelvilleneuve commented 12 months ago

@amaurydubot l'usager a bien une seule participation marquée "noshow", mais le rdv possède 25 participations dont plusieurs en seen. Donc le statut du RDV est seen.

Sauf que dans l'export c'est bien le statut du RDV qu'on affiche et non celui de la participation.. On pourrait afficher le statut de la participation mais je pense qu'il y a plusieurs questions à se poser :

Qu'en penses-tu ? cc @aminedhobb peut-être as-tu plus d'info sur pourquoi on choisi d'afficher le statut du RDV vs la participation ?

aminedhobb commented 12 months ago

Je laisserai @qblanc confirmer mais je pense qu'on avait juste pas fait les modifications nécessaires pour prendre en compte la participation à la place du rdv.

qblanc commented 12 months ago

Pourquoi y'a t-il 25 participations ? Est-ce un usecase habituel ?

J'imagine qu'il s'agit d'un rdv collectif. La table participations est en réalité plus ou moins une table de jointure rdv_users

Est-ce qu'on veut vraiment changer la logique d'affichage du statut (c-a-d utiliser la participation et non le rdv comme source d'info) uniquement pour cette ocurrence de bug ?

Effectivement comme le dit Amine, les infos de la participation sont normalement toujours plus pertinentes que celles du rdv, du point de vue du user. Donc c'est probablement une erreur ou un oubli de ma part.

Michaelvilleneuve commented 12 months ago

Ok je vais corriger dans ce cas et utiliser le statut de la participation dans l'export, merci pour votre aide 🙏

amaurydubot commented 11 months ago

cc @Michaelvilleneuve es-tu intervenu sur ce ticket depuis la semaine dernière? Sinon on va le reprio demain.

Michaelvilleneuve commented 11 months ago

@amaurydubot Non désolé pas encore, ok pour le reprio semaine prochaine, j'essai le prendre demain sinon 🙏