ETQ admin je peux activer la délivrance des attestations v2 (sous feature flag)

[colinux]

• Je n'ai pas le feature flag v2, j'ai une attestation v1 délivrée
• J'ai une v1 publiée, feature flag v2, brouillon v2 : la v1 continue d'être délivrée
• J'ai une v1 publiée, feature flag v2, je publie une v2 : la v1 est supprimée et la v2 est délivrée
• Je n'ai pas de v1, feature flag v2 : je peux préparer et publier une v2 sans passer par la v1

Comme pour les champs du formulaire, les modifications sur l'attestation sont en autosave et doivent être explicitement publiés pour être répercutés sur l'attestation délivrée. Le nouveau bandeau sticky suggéré par @marleneklok sera d'ailleurs repris sur l'écran du formulaire. La notion d'activation de la délivrance est indépendante.

• Ma démarche est en brouillon: j'édite directement l'attestation qui sera délivrée lorsque la démarche sera publiée
• Ma démarche est publiée, j'édite un brouillon de l'attestation :
  • ce brouillon ne sera publié qu'à la publication des modifications
  • s'il y avait une attestation déjà publiée, elle continue d'être délivrée tant que le nouveau brouillon n'est pas publié

(déso pour l'aperçu, mon install python est pétée ce matin)

[codecov]

Codecov Report

Attention: Patch coverage is 94.50549% with 5 lines in your changes missing coverage. Please review.

Project coverage is 81.05%. Comparing base (d7f953f) to head (617c0e4).

Files	Patch %	Lines
...104_backfill_attestation_template_v2_as_draft.rake	50.00%	3 Missing ⚠️
...pp/controllers/instructeurs/dossiers_controller.rb	0.00%	1 Missing ⚠️
app/services/weasyprint_service.rb	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10465      +/-   ##
==========================================
+ Coverage   81.02%   81.05%   +0.03%     
==========================================
  Files        1220     1223       +3     
  Lines       25845    25901      +56     
  Branches     4667     4682      +15     
==========================================
+ Hits        20940    20995      +55     
- Misses       4905     4906       +1     

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

[LeSim]

J'ai hate !

[LeSim]

je comprends qu'on ait besoin de default = published pour garder les v1 publié mais pour toutes les nouvelles attestation_template, il ne faudrait pas avoir default = draft ?

[colinux]

non car si la démarche n'est pas encore publiée, on créé une attestation publiée (C'est une reco de Marlène). De toute façon l'attribut est set au niveau du build de l'attestation dans le controller

[LeSim]

je ne comprends pas l’intérêt d'en faire un méthode d'instance plutôt qu'une méthode de classe.

[colinux]

J'ai changé pour du statique. C'était une habitude (en particulier tirée des patterns des services pour faraday, de la façon dont peuvent évoluer les services, ajouts de middlewares, etc…).