Skip to content

Create and edit items in the history table

Nicolas Bouilleaud requested to merge inline-forms into main

Ajout et edition des CA, Frais et Salaires directement depuis la table.

Notes en vrac et trucs améliorables, mais je crois que ça suffit pour cette MR:

  • Le code est répétitif; ça vaudrait peut-être le coup de DRY, mais d’un autre côté, ça créerait un controller ultra-générique. 🤷
  • Il y a aussi un peu de i18n manquante, mais ça va dans le même esprit de “truc générique”.
  • On pourrait utiliser la même logique pour les objets non “activité mensuelle”, les associés/modèles de répartition/bénéfices.
    • Pour le sport, on pourrait faire en sorte que la page edit fonctionne aussi bien en modal (via turbo) que en html dans un layout normal.
  • Il faudrait faire un petit controller stimulus au lieu du code js de bulma.
  • L’ajout/modification/suppression redirigent vers la page de l’associé (la table), plutôt que de faire une modification en turbo. Si on veut faire ça, il faut clarifier les dépendances entre les objets en base pour savoir ce qui doit être mis à jour et turbo-remplacé.
    • Dans d’autre projets où on fait du turbo broadcast, c’est directement dans les hooks des modèles. Si on veut mettre à jour les objets modifiés sans faire de stream, on doit récupérer la liste des records modifiés en cascade par la transaction. Je crois que transaction.records fonctionnerait, mais c’est un peu expérimental.

Merge request reports