-
Notifications
You must be signed in to change notification settings - Fork 0
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
Permet le téléchargement de l'attestation des déclarations en abandon #1393
Conversation
…re-abandon status
except Exception as e: | ||
logger.error(f"Error obtaining certificate date for declaration {declaration.id}") | ||
logger.exception(e) | ||
date = declaration.creation_date | ||
|
||
try: | ||
submission_actions = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Juste pour info : Je commence à me servir assez souvent dans metabase des catégories d'actions (submissions_action, ongoing_instruction_action, etc).
Je me demande si on ne devrait pas définir ça à un plus haut niveau à un moment donné
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah et je viens de capter du coup que ma catégorie submission n'est pas tout à fait exacte à cause du visa qui peut envoyer des AWAITING_INSTRUCTION
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tout à fait, je pense que par la suite on pourra créer des groupes d'actions dans le modèle Snapshot pour nous faciliter la vie
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Le status.OBSERVATION ne génère pas de certificat ici. Et pourtant le bouton de téléchargement va s'afficher.
…ns that used to be in observation
Merci de ta vigilance sur ce comment @pletelli ! Dans le commit 66b568e j'ai réglé la situation des déclarations En deux mots, le passage en abandon ne génère pas de Snapshot (car pas lié à une action usager). Donc on peut regarder le statut du dernier snapshot pour savoir si précédemment la déclaration était en observation (pas de certificat) ou objection (certificat). BackendCôté backend c'est assez simple, dans la view on regarde dans FrontendDans le front seulement le composant De cette façon Pour aprèsÀ tenir en tête que dans ces views on commence à faire plusieurs appels API en parallèle (déclaration, info de la compagnie, snapshots, etc). Par la suite p-e ça vaudrait la peine d'enrichir le serializer de base pour qu'il contienne toutes ces infos dès le départ pour diminuer le nombre de requêtes. |
Closes #1266
Contexte
Ce n'était pas possible de télécharger l'attestation des déclarations en abandon. L'état pré-abandon (observation ou objection) pouvait être utilisé pour avoir l'attestation.
Scope
Pour le calcul du template on utilisait seulement le statut de la déclaration. Cette PR fait un calcul supplémentaire si la déclaration se trouve en abandon :
Bug fixes
J'en ai profité pour régler un petit bug que j'ai remarqué concernant les dates de l'attestation. Précédemment on calculait ces dates à partir du statut des snapshots, considérant par exemple qu'un snapshot avec le statut
AWAITING_INSTRUCTION
venait forcément d'une soumission de l'usager. Ceci n'est pas le cas car un refus de visa peut aussi générer un snapshotAWAITING_INSTRUCTION
, néanmoins ce n'est pas celui là qu'on doit utiliser pour la date.J'ai donc ajouté des filtres pour l'action du snapshot, histoire d'ignorer les refus de visa (ce sont les changements de la fonction
get_context
)Démo
Screencast.from.2024-12-17.12-34-07.mp4