Skip to content

Préavis - Ajout de la mention 'zéro' dans les mails et sms si pno zero#4981

Draft
n0izn0iz wants to merge 2 commits intomasterfrom
norman/preavis-zero-mention
Draft

Préavis - Ajout de la mention 'zéro' dans les mails et sms si pno zero#4981
n0izn0iz wants to merge 2 commits intomasterfrom
norman/preavis-zero-mention

Conversation

@n0izn0iz
Copy link
Copy Markdown

@n0izn0iz n0izn0iz commented Apr 6, 2026

Fixes #3735

  • Ajout d'un utilitaire is_prior_notification_zero dans le pipeline
  • Ajout de la mention zéro dans le sms si PNO zero
  • Ajout de la mention ... - préavis zéro dans l'objet du mail et le titre html si PNO zero

WIP, pas testé

Comment tester le pipeline de notif sms et mail en local?

@tristanrobert
Copy link
Copy Markdown

tristanrobert commented Apr 6, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Signed-off-by: Norman <norman@samourai.coop>
@n0izn0iz n0izn0iz force-pushed the norman/preavis-zero-mention branch from 96bc156 to 53daf36 Compare April 6, 2026 11:56
@n0izn0iz n0izn0iz changed the title Préavis - Ajout de la mention 'zéro' si pno zero dans les mails et sms Préavis - Ajout de la mention 'zéro' dans les mails et sms si pno zero Apr 6, 2026
Signed-off-by: Norman <norman@samourai.coop>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 6, 2026

@louptheron
Copy link
Copy Markdown
Collaborator

@n0izn0iz Vous avez pu faire un tour avec Vincent sur les tests ou je te détaille ici ?

VincentAntoine added a commit that referenced this pull request Apr 10, 2026
En essayant de faire tourner les tests du pipeline dans le cadre de la
PR #4981, j'ai eu une erreur `Timed out while attempting to connect to
ephemeral Prefect API server.`

En observant les logs prefect, on peux voir que la lib `greenlet` est
manquante.

Après investigation avec @VincentAntoine , on a vu que `greenlet` était
pull par `sqlite` normalement, mais pas sur mac arm64 à cause d'une
clause "markers", on a donc décidé d'ajouter `greenlet` en dépendance de
dev pour régler le problème.
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.

Préavis – Ajouter la mention "préavis zéro" dans les éléments diffusés

3 participants