Batch / traitement multiple

[mfo]

le plan

• une v0 ; basic mais fonctionnel (une colonne vertebrale qui fonctionne sur archive/accepter pour derouler le reste)
• une v1 ; fluide pour accepter les contributions d'@akarzim, moins inconfortable pour les instructeurs (cf: le point sur empecher de tenter une action qui va foirer : ce qui peut arriver en prod aujourd'hui)
• une v2 ; on couvre les cas d'usage important (donc contributions acceptée)

TODO: ui/ux/product

/!\ attention se sont des protos de tech /!\

Concernant le lock sur les dossiers, trop de problèmes possible donc on part sur le strict minimum

détail sur ça : aujourd'hui les instructeurs peuvent deja essayer de passer un dossier en construction quand un autre instructeur l'a fait de son coté. c'est un problème global de l'app, ça nous semble revenir au sujet du traitement par plusieurs instructeurs/de la collaboration en live. Intégrer cette question ici nous semble etre une mauvaise idée ; c'est ouvert a discussion, juste notre parti pris avec @lisa-durand

• TODO: une alerte en haut de page de vue d'un dossier (sans bloquer les actions, les problème arriveront de toute manière, ce qu'y nous semble compter est de gerer ces problèmes)
• un indicateur de traitement multiple sur la vue d'une liste de dossier codé

Concernant l'interface de selection des traitement multiples

• on part sur un bricolage entre @lisa-durand & @mfo et on espere un retour sur le traitement multiple par dsfr

  codé :

rechercher d'idée UX / contrainte

v0

• [ ] comment saisir la justification ? une modal pour la justification / PJ : https://ux.stackexchange.com/questions/18332/best-way-to-do-bulk-edits-on-a-table
• [ ] quel type de tableau et d'action ? CF redemander le tableau avec action multipile au DSFR
• [ ] comment avertir l'utilisateur qu'un dossier est en attente de traitement multiple

v1

• [ ] comment avertir l'utilisateur qu'il ne peut pas interagir avec un dossier en attente de traitement multiple [a la limite, v0 pas important, juste ca doit pouvoir planter gracieusement car ca peut et va arriver]

TODO: tech

v0 prod (juste archiver)

• [x] poc minimalist du traitement en masse pour archiver

• [x] poc UI pour archiver des dossiers en masse

• [x] point d'etape avec l'equipe sur l'archi/le design pour decider si on est dans la bonne direction.

• [x] ressortir le form avec les form=id

• [x] Confirmer avec olivier cette idée de soumettre le form au moment du select ?

• [x] ajouter le message d'info de traitement en masse sur la vue instructeurs/dossiers#show

• [x] ajouter un before_action sur les actions de instructeurs/dossiers_controller.rb pour empecher d'executer une action (accepter etc... pas possible quand il y a un batch en cours sur un dossier) : extend_conservation, follow, unfollow, archive, unarchive, passer_en_instruction, repasser_en_construction, repasser_en_instruction, terminer, restore

• [x] eviter le rescue all ; et recatcher au niveaud u job

• [x] reprendre le code d'atomic_append dans les jobs ?

• [x] question sur l'intégrité des données / BatchOperation.has_many :dossiers : faire une requete qui garantisse qu'on utilise en ensemble de dossier compatible a prioris avec l'operation

• [x] refaire la requete safe query au moment du job

v1

• [ ] quid des api?
• [ ] quid de l'auto archive?
• [ ] poc pour une interface unie entre archive/accepter de sorte a accepter facilement les contributions de l'adullact
• [ ] poc minimalist du traitement en masse pour accepter (donc un systeme de modal supportant motivation & pj). Ce poc doit permettre une contribution facile par l'adullact (en gros, on touche a quelle partie de l'app pour supporter une nouvelle action de masse)

---

Ideation a 4

Just make it batch (and shine)

Proposition

on rajoute une relation belongs_to sur les dossiers qui est lié à un object batch qui décrit l'opération

On veut permettre un nombre arbitraire de dossier

le batch n'est pas atomique (certains dossiers peuvent échouer sans bloquer les autres) -> mais il faut pouvoir avertir l'instructeur

pour les dossiers en cours de traitement, on bloque les changements d'état et toutes les opérations (suppression, archivage ...)

Nouveaux  objects : 
Batch < ApplicationRecord  (attributes: )
has_many_attached :blobs
* ? operation_name: :enum, archive
* question
* failed_dossier_ids: ?
* success_dossier_ids: ?
* dossier_ids: (for undo)
* payload: ex: {motivations, instructeur_id, blob_id}
* created_at/updated_at
* run_at
* finished_at

BatchOperation (archive, )
ArchiveOperation.new(_1).process
dossiers.each { ArchiveOperation.new(_1).process_later }

dossiers.each { _1.passer_en_instruction }

BatchOperation.new(ids, -> (d) d.passe_en_instruction)

## lock de l'interface/dossier lorsqu'un batch est en cours de traitement
Chemins ou il faut bloquer les actions visuellement car un batch est en cours de traitement / les locks

Il y a deux cas

sur des tableaux ex: un instructeur recharge la page après avoir lancé un batch 
    app/views/instructeurs/procedures/_dossier_actions.html.haml

sur une dropdown ex: un instructeur essaie de changer l'etat d'un dossier qui est deja dans un batch (lancé en parallele par un autre instructeur)
    app/views/instructeurs/dossiers/_state_button.html.haml

"conclusion" : il nous faudrait une alerte en haut de page de vue d'un dossier pour expliciter le cas de boutons desactivé

## Reporting batch bug/success

Inversement, il y a aussi le cas ou un instructeur essaie d'executer un batch sur une liste de dossier, mais dans laquelle un des dossier a changé d'etat (ex: par un autre instructeur), il faut gerer le reporting en fin de batch

[dzc34]

voir aussi #7805

[mfo]

voir aussi #7805

@dzc34 / @akarzim - en effet c'est notre base de reflexion vous avez fait un super boulot :-) PS: j'en ai profité pour répondre aux question , certains point seraient a confirmer

[dzc34]

voir le commentaire de suivi Adullact de la liste des actions déjà implémentées, à implémenter ou qui ne seront pas implémentées : --------> https://github.com/demarches-simplifiees/demarches-simplifiees.fr/issues/7805#issuecomment-1386967187