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

feat(dossier): add revision to traitements #11291

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tchak
Copy link
Member

@tchak tchak commented Feb 5, 2025

Maintenant que le rebase est automatique, au support, on me fait remarquer que ce serait intéressant d'avoir accès à la révision sur laquelle un dossier a été déposé. Il y a plusieurs cas d'usages possibles :

  • exposée l'information dans l'API
  • afficher l'info aux instructeurs : ce dossier a été déposé par le porteur sur la version X.XX de la démarche publiée le JJ/MM/AAAA à HH:MM
  • dans l'onglet demande, pouvoir basculer entre les réponses à la version de dépôt et la dernière version

Le souci, c'est qu'aujourd'hui, on perd cette info. Je propose déjà d'enregistrer l'info de la révision sur chaque traitement. J'ai aussi exposé l'information dans l'API parce que c'est facile et que ça rend service aux "fonds vert".

Pour les données historiques, si on avait la motivation, je pense qu'il serait possible de mouliner les dossier_operation_logs et de récupérer l'info. À voir si on le fait dans un second temps.

@tchak tchak force-pushed the feat-add-traitements-revision_id branch from e40fa33 to 7352371 Compare February 5, 2025 10:03
Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.43%. Comparing base (54b5401) to head (52f0b99).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #11291   +/-   ##
=======================================
  Coverage   84.43%   84.43%           
=======================================
  Files        1211     1211           
  Lines       26688    26692    +4     
  Branches     5038     5038           
=======================================
+ Hits        22533    22537    +4     
  Misses       4155     4155           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tchak tchak force-pushed the feat-add-traitements-revision_id branch from 7352371 to 52f0b99 Compare February 5, 2025 14:30
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