Skip to content

feat: traçabilité de l'envoi des factures dans la tréso - #2396

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-en-gros-le-process
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-en-gros-le-process

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 14, 2026

Copy link
Copy Markdown
Member

Description

Le back-office facture ne laissait aucune trace de l'envoi des factures : impossible de savoir si une facture avait été envoyée, par qui et quand. Ce PR ajoute cette traçabilité et en copie le mail aux sponsors.

Changes

  • Le mail d'envoi de facture met désormais sponsors@afup.org en copie
  • Chaque facture affiche un statut d'envoi avec la date et l'auteur de l'envoi au survol
  • Nouveau bouton pour marquer une facture comme envoyée manuellement
  • Migration de base de données : date d'envoi et auteur de l'envoi sur les factures

How to test

  1. Depuis la liste des factures, cliquer sur « Envoyer la facture par mail » : le statut passe à « Envoyé », avec la date et l'auteur visibles au survol
  2. Cliquer sur le bouton de marquage manuel : la facture est marquée « Envoyé » sans envoi de mail
  3. Lancer la migration Phinx pour ajouter les colonnes date_envoi et envoye_par à afup_compta_facture

Prises d'écran

Nouveau rendu de la page
image

Survol du nouveau bouton pour forcer l'envoi
image

La facture a été marquée comme envoyée
image

Survol du statut "envoyé" de la facture
image

@Korbeil
Korbeil force-pushed the afup-en-gros-le-process branch from 4161fba to da89922 Compare September 14, 2026 20:32
@Korbeil
Korbeil force-pushed the afup-en-gros-le-process branch from da89922 to a003837 Compare September 14, 2026 21:02
@vgreb

vgreb commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

On pourrait utiliser la classe AppBundle\AuditLog\Audit plutôt que recréer un système spécifique aux factures. Qu'en penses tu ?

@Korbeil
Korbeil marked this pull request as ready for review September 14, 2026 21:15
@Korbeil

Korbeil commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

On pourrait utiliser la classe AppBundle\AuditLog\Audit plutôt que recréer un système spécifique aux factures. Qu'en penses tu ?

J'avoue que je connais pas tout dans le code actuel, mais je vais voir ce que ça donne avec ça ! 👍

@Korbeil

Korbeil commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

On pourrait utiliser la classe AppBundle\AuditLog\Audit plutôt que recréer un système spécifique aux factures. Qu'en penses tu ?

@vgreb je vois qu'il n'y a pas l'id de l'entité source du log dans la table de logs, du coup le seul matching que je peux faire c'est via le nom de la facture (par exemple) ça veut dire faire un LIKE dans la table log, niveau perfs je pense que c'est vraiment pas top 😬
J'avoue que je serais plutôt pour garder les infos dans la table Invoice, mais on peut doubler ça dans les logs histoire d'avoir un suivi là-bas aussi, tu en penses quoi ?

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.

2 participants