Skip to content

♻️ refactor(Accounting): passe de refacto finale et documentation de la comptabilité (#362, livraison 10) - #508

Merged
ronan-develop merged 5 commits into
mainfrom
refactor/#362-passe-finale
Oct 17, 2026
Merged

ronan-develop merged 5 commits into
mainfrom
refactor/#362-passe-finale

Conversation

@ronan-develop

Copy link
Copy Markdown
Owner

Livraison 10 de #362 : passe de refacto (checklist de la colonne Refacto : DRY, SRP, KISS) et documentation.

  • DRY : la saisie d'un mois AAAA-MM (4 copies), le montant en texte (4 copies), le libellé de mois et le nettoyage du nom de fichier (relevés et documents de groupe, qui gagnent le retrait des caractères de format comme l'inversion du sens d'écriture) sont chacun à un seul endroit, avec leurs tests.
  • JS : l'envoi des formulaires (désactivation, erreurs par champ, réactivation) est factorisé dans form-feedback.js et testé ; vérifié dans un vrai navigateur (création, erreur 422, échéance, cotisation, mois sans mouvement).
  • Affichage : les outils de la comptabilité sont une grille de boutons (ils se chevauchaient sur mobile), l'état d'une tâche est séparé de son libellé.
  • Doc : vue d'ensemble de la comptabilité (carte du code, pages et API, règles, tests de sécurité, exploitation, limites connues), copie hors serveur des fichiers dans commands.md, index de CLAUDE.md.

Refs #362 (reste : livraison 7, import CSV, en attente d'un exemple de fichier de la banque).

PHPUnit 3277 OK, npm 395 OK, PHPStan, Psalm taint, php-cs-fixer, budgets OK.

@ronan-develop
ronan-develop merged commit b181b23 into main Oct 17, 2026
4 checks passed
@ronan-develop
ronan-develop deleted the refactor/#362-passe-finale branch October 17, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant