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

Créer une alerte mail lorsque la case "brouillon" a été cochée pour les tronçons #2904

Closed
numahell opened this issue Jan 18, 2022 · 7 comments
Assignees

Comments

@numahell
Copy link

No description provided.

@camillemonchicourt
Copy link
Member

Email du destinataire configurable.
Si non renseigné, pas d'alerte email pour ceux qui ne souhaiteraient pas utiliser ce système d'alertes emails.
A documenter.

@numahell
Copy link
Author

numahell commented May 3, 2022

Idée : créer un paramètre GROUPE_GESTION ou GROUPE_VALORISATION avec l'id des groupes concernés, et ainsi gérer les utilisateurs au niveau de l'admin de django.
Inconvénient, il faut créer les utilisateurs dans l'admin de django.

@numahell
Copy link
Author

numahell commented May 3, 2022

En lien avec la #2903

@camillemonchicourt
Copy link
Member

Je ne suis pas certain que les notions de GROUPE_GESTION et GROUPE_VALORISATION soient bien compréhensibles.
Déjà car le terme GROUPE est déjà utilisé pour les groupes d'utilisateurs et cela peut donc apporter de la confusion.
Aussi car les notions de GESTION et de VALORISATION sont parfois floues selon les usages de Geotrek.

Mais c'est vrai que ce serait certainement plus pertinent de s'appuyer sur les utilisateurs existants dans Geotrek pour définir à qui envoyer les emails, plutôt que de créer des paramètres avec des emails à chaque fois (comme on a actuellement ADMINS et MANAGERS). Quoique dans notre cas au PNE, pour ces 2 paramètres ils ne sont pas liés à des emails de personnes, mais à des listes de diffusion générique.

Je n'ai plus en tête comment les emails à notifier dans le mode Workflow de Suricate, mais il y a peut-être une approche globale à clarifier, homogénéiser ?

@numahell
Copy link
Author

Dans Suricate, c'est les emails des managers qui sont utilisés. Je pense que je vais faire de même, et ensuite on peut réfléchir plus tard dans une autre évolution comment relier aux groupes de django.

@camillemonchicourt
Copy link
Member

Fait dans la 2.86.0 avec l'ajout d'un paramètre ALERT_DRAFT (à FALSE par défaut).

Par contre, j'ai pas compris à quoi correspond MAIL_MANAGERS à qui les emails sont envoyés ?
C'est un paramètre où l'on renseigne des emails ?
Je n'en vois pas de trace.

@numahell
Copy link
Author

C'est un problème dans la doc de Geotrek, c'est en fait MANAGERS. Si ce paramètre n'est pas renseigné, c'est ADMINS qui est utilisé : https://docs.djangoproject.com/en/3.2/ref/settings/#std-setting-MANAGERS

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

No branches or pull requests

2 participants