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

ETQ Administrateur, je souhaite personnaliser les mails de notification "brouillons", ... comme pour les autres mails d'une démarche #7776

Open
adnan-el-moussawi opened this issue Sep 19, 2022 · 1 comment

Comments

@adnan-el-moussawi
Copy link

adnan-el-moussawi commented Sep 19, 2022

Actuellement

Dans l'espace "Administrateur" (admin/procedures/[identifiant]/mail_templates), on peut configurer uniquement les mails relatifs à l'état d’avancement d'un dossier (déposé, en instruction, accepté, refusé, sans suite) :
def mail_templates [ procedure.initiated_mail_template, procedure.received_mail_template, procedure.closed_mail_template, procedure.refused_mail_template, procedure.without_continuation_mail_template ]

image
image


Comportement attendu

Rendre configurable les autres mails de notification relatifs à un dossier :

  • nouveau brouillon
  • brouillon non envoyé
  • repasser en construction
  • repasser en instruction
  • etc.

image

adnan-el-moussawi added a commit to adnan-el-moussawi/demarches-simplifiees.fr that referenced this issue Sep 20, 2022
adnan-el-moussawi added a commit to adnan-el-moussawi/demarches-simplifiees.fr that referenced this issue Sep 20, 2022
…ion", ''revert_to_instruction" mails

- Add Models and Views for ''new_draft", ''revert_to_construction", ''revert_to_instruction" mails
- Update MailTemplatesController

Refs : demarches-simplifiees#7776
adnan-el-moussawi added a commit to adnan-el-moussawi/demarches-simplifiees.fr that referenced this issue Sep 21, 2022
…tion", ''revert_to_instruction" mails

- Ajout de la structure des tables de données pour les nouveaux mails

Refs: demarches-simplifiees#7776
@dzc34
Copy link
Collaborator

dzc34 commented Sep 27, 2022

Visio du 21/09/2022 avec @adnan-el-moussawi, @tchak et  @krichtof

Ci-dessous, mes notes sur nos échanges (hors debug technique)

Refacto

Refacto souhaitable, idéalement en amont :

  • regrouper les fichiers de mails dans les mêmes dossiers
  • DB - Tables de personnalisation des mails à transformer en 1 seule table (1)

(1) utiliser un STI (Single table inheritance)

  • 1 class de parent = la table commune
  • classe enfante par type de mail ---> et rails fait tout

Proposition de découpage / organisation de la contribution + refacto

Idéalement :

  • 1 PR pour le refacto --> 1 seul table de personnalisation des mails / ...
  • 1 PR pour ajouter les nouveaux emails customisable
  • 1 ticket dédié pour le mail "revert_to_construction" qui n'existe pas actuellement

Solution alternative :

  • x PR = 1 PR pour chaque email email customizable
  • 1 PR pour le refacto --> 1 seul table de personnalisation des mails / ...
  • 1 ticket dédié pour le mail "revert_to_construction" qui n'existe pas actuellement

adnan-el-moussawi added a commit to adnan-el-moussawi/demarches-simplifiees.fr that referenced this issue Oct 27, 2022
…ion", ''revert_to_instruction" mails

normalisation des noms des fichiers, des classes, des modèles et des tables de données (en français)

Refs: demarches-simplifiees#7776
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