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

[TRA 15352] Remonter le VHU en situation irrégulière (sans émetteur TD) dans l'onglet À collecter #3792

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

silto
Copy link
Collaborator

@silto silto commented Nov 29, 2024

Objectif

Les BSVHU peuvent avoir un émetteur en situation irrégulière. Dans ce cas, il est possible que l'émetteur n'ait pas de compte TD, ou qu'il ne soit même pas une entreprise. Dans ces cas, il faut que le bordereau apparaisse dans l'onglet "A collecter" du transporteur, car le transporteur sera le premier acteur à signer le bordereau.

Tech

La catégorie "n'a pas d'émetteur inscrit sur TD" n'est pas directement "lisible" sur le bordereau pour l'indexation :

  • le flag emitterIrregularSituation correspond à un émetteur en situation irrégulière, mais qui peut quand même être inscrit sur TD
  • le flag emitterNoSiret correspond à un émetteur sans SIRET (un particulier), mais n'est pas suffisant car un émetteur avec SIRET peut ne pas être inscrit sur TD

2 solutions étaient donc possibles :

  • ajouter une récupération de company au moment de l'indexation
    • allourdi la logique d'indexation
    • oblige à passer les méthodes d'indexation en async, ce qui n'est pas le cas actuellement
  • ajouter un flag sur le bordereau et l'utiliser au moment de l'indexation
    • si l'établissement s'inscrit sur TD avant que le transporteur ait signé, le bordereau ne disparaît pas de l'onglet "A collecter"

J'ai choisi la solution 2 car moins disruptif de la logique d'indexation pour un bordereau "mineur" qu'est le BSVHU. De plus, le bordereau apparaît quand même dans le bordereau de l'émetteur si il s'inscrit après la création du bordereau, donc pas bloquant.


  • Mettre à jour la documentation
  • Mettre à jour le change log
  • Documenter les manipulations à faire lors de la mise en production (sur le ticket Favro de release)
  • S'assurer que la numérotation des nouvelles migrations est bien cohérente
  • Informer le data engineer de tout changement de schéma DB

Copy link

sonarcloud bot commented Nov 29, 2024

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.

1 participant