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

Améiorer la situation sur les traitements/ÉvènementPhaseDossier #185

Open
DavidBruant opened this issue Feb 27, 2025 · 0 comments
Open

Comments

@DavidBruant
Copy link
Collaborator

Pour contexte, initialement, on créait un ÉvènementPhaseDossier par "traitement" sur un dossier DS
et on a nos propres ÉvènementPhaseDossier (surtout pour la phase "Étude recevabilité DDEP" pour le moment)

Mais, on a découvert que DS, via son API expose des "traitements" qui ne correspondent pas vraiment à des changements de phase. Quand un dossier est corrigé, un "traitement" est ajouté (alors que ça ne correspond pas à un changement de "state" DS et donc phase Pitchou)
Ça créé une situation où visuellement, on voit un dossier qui est déjà en accompagnement amont passer en accompagnement amont 5 fois d'affilée, ce qui n'a aucun sens

Après discussion avec l'équipe de DS, il semble qu'une manière de détecter ça, c'est que les "faux traitements" ont un emailAgentTraitant: null

Dans #184 , on récupère les emailAgentTraitant et on les stocke en base de donnée dans un colonne DS_emailAgentTraitant. Et ensuite, on n'affiche jamais les ÉvènementPhaseDossier qui ont DS_emailAgentTraitant IS NULL and cause_personne IS NULL

Ça pollue un peu la BDD, mais c'est plus prudent

Idéalement, la table évènement_phase_dossier ne devrait pas avoir ces lignes.
Ptèt qu'un jour on arrivera à faire mieux

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

No branches or pull requests

1 participant