ETQ instructeur, je peux renommer le contenu de mon export zip #10217

krichtof · 2024-03-26T10:42:27Z

Voici ce que fait cette PR :

elle apporte une modification dans l'export zip classique (qui n'utilise pas de modèle d'export) : le répertoire à la racine dans le zip n'est plus de la forme dossiers_<nom demarche>_<date-heure>, ce répertoire est renommé export
il est possible désormais pour l'instructeur de créer, modifier ou supprimer un modèle d'export dans lequel il pourra renommer le contenu de l'export (pour l'instant uniquement au format zip). Il sera éditable par l'ensemble des instructeurs du groupe
- c'est notamment les pièces justificatives qui peuvent être renommées (uniquement celles de la révision active)
- il est possible d'utiliser des tags (liés au dossier) dans les noms des fichiers. La liste des tags disponibles est restreinte pour des raisons d'UI
lorsqu'on souhaite créer un export, il est désormais possible de choisir un modèle d'export précédemment créé (ceci est possible depuis le dropdown Télécharger tous les dossiers
les modèles d'export déjà créés sont visibles dans la page Voir les exports

Quelques screenshots :

LeSim · 2024-03-28T17:19:41Z

config/locales/models/export_templates/en.yml

+        tiptap_default_dossier_directory: Nom du répertoire
+        tiptap_pdf_name: "Nom de l'export"


a anglaitisé (quoi, je parle pas bien ?)

Copy/paste is hell !

LeSim

Franchement le rendu est vraiment super top. Bravo !

J'ai repéré un ptit glitch quand je le fais tourner sur une procédure avec de long nom d image, le numéro du dossier se retrouver repousser loin ...

J'ai l'impression qu'on ne peut pas choisir les pjs qu'on veut exporter. Ca sera pour une prochaine PR ?

LeSim · 2024-03-28T17:20:12Z

config/locales/models/export_templates/fr.yml

+          name: "Le nom sera visible par vous et les autres instructeurs pour générer un export"
+          tiptap_default_dossier_directory: "Comment souhaitez-vous nommer le répertoire contenant les documents d'un dossier ?"
+          tiptap_pdf_name: "Comment souhaitez-vous nommer le fichier pdf qui contient toutes les réponses de l'usager ?"
+        name: "Nom"


Suggested change

name: "Nom"

name: "Nom du modèle"

Corrigé ici : 9f023f9

LeSim · 2024-03-28T17:21:39Z

config/locales/models/export_templates/fr.yml

+          tiptap_pdf_name: "Comment souhaitez-vous nommer le fichier pdf qui contient toutes les réponses de l'usager ?"
+        name: "Nom"
+        tiptap_default_dossier_directory: Nom du répertoire
+        tiptap_pdf_name: "Nom de l'export"


Suggested change

tiptap_pdf_name: "Nom de l'export"

tiptap_pdf_name: "Nom du dossier au format pdf"

LeSim

J'ai déjà dit que l aperçu en reprenant les valeurs d un dossier était extrêmement stylé ?

Alors pleins de retours, mais c'est parce que c'est génial comme fonctionnalité 🚀

Important

Je reviens sur ma position, je pense que la date est importante dans le nom du fichier zip -> export_29_03_2024 pour distinguer les différents fichiers.
stp stp stp stp pouvoir décocher des pièces jointes stp stp stp stp (voir mm avoir tout décoché par défault et avoir un bouton pour tout cocher (voir mm avertir l'instructeur que s'il coche trop de pj ca va prendre trop de temps (ca fait beaucoup de (), non? ))).

Moins important

UI
- je n'ai pas trouvé immédiatement comment modifier mon modèle d'export, peut être rajouter un lien dans les listes déroulantes ?
- trouver un moyen de désactiver la correction orthographique sur les inputs de pjs car c'est tout rouge chez moi ( un truc comme editorProps: { attributes: { class: 'fr-input', spellcheck: true } } dans tiptap/editor.ts:126 mais faudrait variabiliser.
- en haut de la page configuration des exports, j'essayerai bien aligner le haut du bloc aperçu en face du champ nom
fonctionnalités
- je me demande si on ne pourrait pas proposer un bouton pour essayer avec un seul dossier pour que l'admin se rendent compte sans avoir a générer un export complet.

LeSim · 2024-03-29T08:37:34Z

app/views/instructeurs/export_templates/_form.html.haml

+              Cette page permet d'éditer un modèle d'export et ainsi personnaliser le contenu des exports (pour l'instant,
+              uniquement au format zip). Ainsi, vous pouvez notamment normaliser le nom des pièces jointes.
+              Essayez-le et donnez-nous votre avis
+              en nous envoyant un email à #{mail_to(CONTACT_EMAIL, subject: "Editeur de modèle d'export")}.


LeSim · 2024-03-29T08:42:55Z

app/components/dossiers/export_dropdown_component/export_dropdown_component.html.haml

@@ -14,3 +14,12 @@
    - menu.with_item do
      = link_to download_export_path(export_format: format), role: 'menuitem', data: { turbo_method: :post, turbo: true } do
        = t(".everything_#{format}_html")
+
+  - unless export_templates.nil?


Suggested change

- unless export_templates.nil?

- if export_templates.present?

Corrigé dans le commit rebasé : f204f73#diff-05bd09b918a8fde624342651354061d91edbb6e8640cbc74f3bb81067e2801bdR18

app/models/export_template.rb

LeSim · 2024-03-29T09:19:58Z

db/migrate/20240130154452_create_export_templates.rb

+      t.string :name
+      t.string :kind


Suggested change

t.string :name

t.string :kind

t.string :name, null: false

t.string :kind, null: false

Corrigé dans le commit rebasé : ea86c04

krichtof · 2024-03-29T10:12:43Z

Franchement le rendu est vraiment super top. Bravo !

Merci aussi à @colinux pour l'aide sur le sujet :)

J'ai repéré un ptit glitch quand je le fais tourner sur une procédure avec de long nom d image, le numéro du dossier se retrouver repousser loin ... !

Fichtre ! Je vais regarder ça

J'ai l'impression qu'on ne peut pas choisir les pjs qu'on veut exporter. Ca sera pour une prochaine PR ?

Effectivement, je n'avais pas noté ce besoin. Peut-être dans une prochaine PR si le besoin nous est remonté ?

krichtof · 2024-03-29T12:32:03Z

* Je reviens sur ma position, je pense que la date est importante dans le nom du fichier zip -> `export_29_03_2024`  pour distinguer les différents fichiers.

Le nom du fichier zip n'a pas changé, la date est toujours là. C'est dans le contenu du zip que ça a changé. Le répertoire racine se nomme désormais export et ne contient plus la date

* stp stp stp stp pouvoir décocher des pièces jointes stp stp stp stp (voir mm avoir tout décoché par défault et avoir un bouton pour tout cocher (voir mm avertir l'instructeur que s'il coche trop de pj ca va prendre trop de temps (ca fait beaucoup de (), non? ))).

Ca te va si on en discute et que ça fasse l'objet d'une autre PR ?

  * je n'ai pas trouvé immédiatement comment modifier mon modèle d'export, peut être rajouter un lien dans les listes déroulantes ?

Cela voudrait dire que pour chaque modèle d'export, dans la liste déroulante on aurait un lien pour générer l'export à partir de ce modèle, et un lien pour éditer le modèle ? Ca risque pas d'être un peu chargé ?

  * trouver un moyen de désactiver la correction orthographique sur les inputs de pjs car c'est tout rouge chez moi ( un truc comme `editorProps: { attributes: { class: 'fr-input', spellcheck: true } }` dans tiptap/editor.ts:126 mais faudrait variabiliser.

Dacodac

  * en haut de la page configuration des exports, j'essayerai bien aligner le haut du bloc aperçu en face du champ nom
    ![Screenshot 2024-03-29 at 09-34-43 demarches-simplifiees fr]

Dacodac

  * je me demande si on ne pourrait pas proposer un bouton pour essayer avec un seul dossier pour que l'admin se rendent compte sans avoir a générer un export complet.

Pourquoi pas. Tu penses que cela apporterait un plus par rapport à la preview ?
Je vote pour que ça fasse l'objet d'une autre PR :)

krichtof · 2024-04-05T09:06:52Z

en haut de la page configuration des exports, j'essayerai bien aligner le haut du bloc aperçu en face du champ nom
![Screenshot 2024-03-29 at 09-34-43 demarches-simplifiees fr]
Dacodac

Fait dans ce nouveau commit @LeSim : 323748b

codecov · 2024-04-24T09:52:32Z

Codecov Report

Attention: Patch coverage is 93.49315% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 80.67%. Comparing base (b3c2b46) to head (0869168).

Files	Patch %	Lines
...ollers/instructeurs/export_templates_controller.rb	84.74%	9 Missing ⚠️
app/models/export_template.rb	91.02%	7 Missing ⚠️
.../controllers/administrateurs/exports_controller.rb	33.33%	2 Missing ⚠️
app/models/champ.rb	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10217      +/-   ##
==========================================
- Coverage   80.80%   80.67%   -0.14%     
==========================================
  Files        1200     1204       +4     
  Lines       25325    25555     +230     
  Branches     4568     4624      +56     
==========================================
+ Hits        20465    20616     +151     
- Misses       4860     4939      +79

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

…r_champ

Co-authored-by: Christophe Robillard <christophe.robillard@beta.gouv.fr>

krichtof force-pushed the 9827-export-rename branch 10 times, most recently from 19cebd8 to f21665e Compare March 27, 2024 13:07

krichtof marked this pull request as ready for review March 27, 2024 13:32

LeSim reviewed Mar 28, 2024

View reviewed changes

LeSim reviewed Mar 29, 2024

View reviewed changes

krichtof force-pushed the 9827-export-rename branch 2 times, most recently from e9635fe to 2ad76f8 Compare March 29, 2024 13:02

LeSim force-pushed the 9827-export-rename branch 2 times, most recently from 19f45f2 to e5eb493 Compare April 24, 2024 09:51

LeSim force-pushed the 9827-export-rename branch from e5eb493 to a438d34 Compare April 24, 2024 13:09

LeSim force-pushed the 9827-export-rename branch 2 times, most recently from b5ff39e to 29b209a Compare May 7, 2024 07:26

LeSim force-pushed the 9827-export-rename branch from d9458bf to b780aac Compare May 15, 2024 08:08

krichtof force-pushed the 9827-export-rename branch from d625283 to c192662 Compare May 17, 2024 13:30

LeSim force-pushed the 9827-export-rename branch 3 times, most recently from d01f875 to 59410b7 Compare May 21, 2024 09:11

LeSim added 16 commits May 23, 2024 10:00

fix: champ.row_index and test pjs_for_champs

6d757db

refactor(pj_service): do not query for pj_index

b656867

refactor(suffix): be consistent with index suffix

5858105

spec(perf): count sql queries for 10 dossiers

fe5c655

refactor: remove target in tags

43fb1dd

refactor: remove unused data

1c0bd3e

refactor: simplify

3af1cee

refactor: rename

e0c867f

refactor: memoize flat_tags

d60e790

refactor(tags_substitution): simplify

4205204

refactor: be explicite with memoization

e8a175d

refactor: DossierPreloader rename includes_for_dossier -> includes_fo…

080bcd8

…r_champ

fix test

fa7d5cf

perf(zip): preload_dossier earlier

9effa9e

perf(spec): new count !

a7e29c4

perf(zip): give champ to avoid seeking stable_id

ca12a56

LeSim force-pushed the 9827-export-rename branch from 4f2355d to 6ce6ce4 Compare May 23, 2024 08:00

LeSim and others added 5 commits May 23, 2024 10:01

perf(preloader): preloader use batch for batches

6184b33

perf(spec): new record !

8628ec1

Co-authored-by: Christophe Robillard <christophe.robillard@beta.gouv.fr>

perf(pj service): compute row_id without extraneous requests

e38999e

spec: fix

d3b7003

fix: remove useless code

4fb03e3

LeSim force-pushed the 9827-export-rename branch from 6ce6ce4 to 5b7f050 Compare May 23, 2024 08:01

spec: test champs_id_row_index

0869168

LeSim force-pushed the 9827-export-rename branch from 5b7f050 to 0869168 Compare May 23, 2024 08:08

tchak approved these changes May 23, 2024

View reviewed changes

krichtof added this pull request to the merge queue May 23, 2024

Merged via the queue into main with commit 2761d18 May 23, 2024
19 checks passed

krichtof deleted the 9827-export-rename branch May 23, 2024 09:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ETQ instructeur, je peux renommer le contenu de mon export zip #10217

ETQ instructeur, je peux renommer le contenu de mon export zip #10217

krichtof commented Mar 26, 2024 •

edited

Loading

LeSim Mar 28, 2024

krichtof Mar 29, 2024

LeSim left a comment

LeSim Mar 28, 2024

krichtof Mar 29, 2024

LeSim Mar 28, 2024

LeSim left a comment

LeSim Mar 29, 2024

LeSim Mar 29, 2024

krichtof Mar 29, 2024

LeSim Mar 29, 2024

krichtof Mar 29, 2024

krichtof commented Mar 29, 2024 •

edited

Loading

krichtof commented Mar 29, 2024

krichtof commented Apr 5, 2024

codecov bot commented Apr 24, 2024 •

edited

Loading

		tiptap_default_dossier_directory: Nom du répertoire
		tiptap_pdf_name: "Nom de l'export"

	tiptap_pdf_name: "Nom de l'export"
	tiptap_pdf_name: "Nom du dossier au format pdf"

	- unless export_templates.nil?
	- if export_templates.present?

ETQ instructeur, je peux renommer le contenu de mon export zip #10217

ETQ instructeur, je peux renommer le contenu de mon export zip #10217

Conversation

krichtof commented Mar 26, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LeSim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LeSim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

krichtof commented Mar 29, 2024 • edited Loading

krichtof commented Mar 29, 2024

krichtof commented Apr 5, 2024

codecov bot commented Apr 24, 2024 • edited Loading

Codecov Report

krichtof commented Mar 26, 2024 •

edited

Loading

krichtof commented Mar 29, 2024 •

edited

Loading

codecov bot commented Apr 24, 2024 •

edited

Loading