ETQ usager la liste des pièces justificatives à fournir inclut les PJ des blocs répétables

[colinux]

(Bug signalé au support)

C'est potentiellement pas super adapté dans les anciennes démarches qui incluent des PJ dans des blocs répétables uniquement pour pouvoir envoyer plusieurs fichiers. Mais ils ont la main :

• pour rédiger une liste manuelle de PJ si la liste automatique ne leur convient pas
• ou pour mettre à jour leur formulaire

EDIT: comme suggéré par @LeSim j'ai ajouté du cache de fragment sur une partie de la description, ce qui accélère le chargement de cette page de 25-30% même sur une démarche toute simple 🎉

APRES

AVANT

[LeSim]

top, une proposition de refactorisation.

Est ce que, de manière opportuniste, on mettrait pas un cache sur cette partie ?

[colinux]

@LeSim j'ai ajouté du cache de fragment sur une partie de la description, ce qui accélère le chargement de cette page de 25-30% en local même sur une démarche toute simple 🎉

J'en ai profité pour mieux cadrer comment tester le cache dans les specs , en passant caching: true aux tests qui en ont besoin https://github.com/demarches-simplifiees/demarches-simplifiees.fr/pull/9747/files#diff-305662962b738b57f5dbf7309597b39164807bcbf1b49a06f43eea90901dbc3d

(Je sens que chaque semaine je vais m'en faire 1 ou 2 car le gain à chaque fois est marqué)

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ ActionView::Template::Error: PG::QueryCanceled: ERROR: canceling statement due to statement timeout (ActionView::Template::Er... Users::CommencerController#commencer View Issue

_{Did you find this useful? React with a 👍 or 👎}