Skip to content

✨ Échange de créneau : prévenir par e-mail le groupe titulaire et le demandeur (#455) - #459

Merged
ronan-develop merged 7 commits into
mainfrom
feat/#455-notifier-echange-creneau
Oct 12, 2026
Merged

ronan-develop merged 7 commits into
mainfrom
feat/#455-notifier-echange-creneau

Conversation

@ronan-develop

Copy link
Copy Markdown
Owner

Closes #455

  • À la création d'une demande d'échange : e-mail à l'adresse de contact du groupe titulaire (groupe demandeur, jour, plage, lien). Le motif saisi par le groupe demandeur n'est jamais repris (texte libre d'un groupe vers un autre).
  • À la réponse : e-mail à la personne qui avait demandé (compte actif), « accepté » ou « refusé » par le groupe titulaire.
  • Envoi après la réponse HTTP, jamais bloquant, échec journalisé avec des identifiants seulement, pas de désabonnement général. Objet nul par défaut dans AvailabilityService.

Refacto

SRP : un notifier à part, une seule dépendance ajoutée au service. KISS : une méthode plate (première version à callables imbriqués remplacée). Factory/Strategy écartés : pas de second cas d'usage.

Vérifications

phpunit complet (2378) et Psalm taint verts ; accès (IDOR) : aucune notification quand la demande ou la réponse est refusée.

@ronan-develop

Copy link
Copy Markdown
Owner Author

Refacto fait dans le ticket (la CI l'a imposé : budgets de taille et de classes par dossier) : ownRequest mutualise le contrôle d'accès du groupe demandeur (modification, annulation) ; SlotRequestRange sort d'AvailabilityService les règles de plage, pures ; les notifiers de réservation et d'échange vivent dans Planning/Notification.

@ronan-develop
ronan-develop merged commit 13a1e04 into main Oct 12, 2026
4 checks passed
@ronan-develop
ronan-develop deleted the feat/#455-notifier-echange-creneau branch October 12, 2026 10:11
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.

✉️ Échange de créneau : prévenir par e-mail le groupe titulaire à la demande et le demandeur à la réponse

1 participant