Skip to content

✨ Annonce de l'appli #B27 à tous les inscrits (#444) - #454

Merged
ronan-develop merged 4 commits into
mainfrom
feat/#444-annonce-aux-inscrits
Oct 11, 2026
Merged

ronan-develop merged 4 commits into
mainfrom
feat/#444-annonce-aux-inscrits

Conversation

@ronan-develop

Copy link
Copy Markdown
Owner

Closes #444

Contenu

  • Gabarit announcement (HTML + texte) : but de l'appli, boutons « Activer mon compte » (Mot de passe oublié) et « Me connecter », mini-tuto en 4 points (planning, créneau libre, échange, messagerie).
  • AnnouncementService : envoi unique aux comptes actifs ; essai à une adresse ; mémoire des comptes servis (storage/announcement-sent.txt, identifiants seulement), donc aucun doublon à la relance et réessai des seuls échecs.
  • bin/send-announcement.php : simulation par défaut, --test=adresse, --send.

Refacto (colonne Refacto lue)

OOP/SRP : service, mémoire et bilan séparés. KISS : analyse des options simplifiée. YAGNI : fichier plutôt qu'une migration. Factory/Strategy : pas de second cas d'usage.

Vérifications

phpunit complet (2369) et Psalm taint verts. Rien n'est envoyé par la PR : l'envoi réel se lance à la main après déploiement.

@ronan-develop
ronan-develop merged commit da18d89 into main Oct 11, 2026
4 checks passed
@ronan-develop
ronan-develop deleted the feat/#444-annonce-aux-inscrits branch October 11, 2026 18:22
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.

✉️ Annonce de mise à disposition de l'appli à tous les inscrits (e-mail + script d'envoi)

1 participant