Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Diffusion des préavis à des destinataires hors unités #3895

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

VincentAntoine
Copy link
Collaborator

@VincentAntoine VincentAntoine commented Nov 22, 2024

Linked issues

Présentation

  • Ajout d'une table pno_extra_subscriptions qui permet d'ajouter des destinataires hors unités aux diffusions de PNO
  • Chaque ligne de cette table comporte
    • un locode de port
    • un pno_type_name (type de préavis)
    • le nom du destinataire
    • son organisation
    • le mode de communication (SMS / EMAIL)
    • son email ou numéro de téléphone
  • Le flow distribute_pnos utilise ces abonnements pour assigner des destinataires supplémentaires, en plus des unités, aux préavis dont le port et le type de préavis correspondent à un abonnement
  • Pour l'instant le besoin existant est de diffuser uniquement les préavis navires tiers à certains destinataires par email. Pour éviter de hard coder ceci j'ai préféré prévoir un système plus souple dans le quel on peut paramétrer des diffusions sur n'importe quel type de préavis et par n'importe quel moyen de communication (SMS ou EMAIL).

Impacts

  • L'administration de ces diffusions hors unité se fera en dur / via Metabase
  • Les messages envoyés seront dans prior_notification_sent_messages au même titre que les messages envoyés aux unités
  • Ils apparaitront donc aussi dans la liste de diffusion des préavis dans l'app

@VincentAntoine VincentAntoine force-pushed the vincent/add_foreign_vessels_pno_distribution branch from 8b14048 to c45e2ed Compare November 25, 2024 15:35
Copy link

sonarcloud bot commented Nov 25, 2024

Quality Gate Passed Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@VincentAntoine VincentAntoine marked this pull request as ready for review November 25, 2024 15:37
@VincentAntoine VincentAntoine self-assigned this Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Permettre la diffusion des préavis à des destinataires autres que les unités
2 participants