gems-uff / sapos

SAPOS main goal is to ease the management of information related to graduate programs such as enrollments, courses, advisement, scholarships, requirements, among others.
http://gems-uff.github.io/sapos/
MIT License
29 stars 14 forks source link

Configuração de Template de Email #358

Closed JoaoFelipe closed 3 years ago

JoaoFelipe commented 3 years ago

Fazendo a issue #347, percebi que diversos novos templates de email precisam ser definidos e configurados para ela:

Para a issue #346 o template de email foi definido na variável CustomVariable.account_email. Posso definir esses templates novos como CustomVariable também, mas a interface de edição não é tão boa (já que variáveis são genéricas) e talvez fiquem muitas variáveis de email dificultando a configuração:

image

Ao invés disso, talvez seja melhor definir uma configuração de template separada baseadas na configuração de notificações:

image

Essa configuração de template de email poderia servir tanto para as notificações hardcoded que estou fazendo agora (#346 e #347) quanto para as configurações do devise. O template também poderia ser usado nas notificações configuráveis para diminuir redundância no controle de redirect_email.

O que acham? Vale a pena investir tempo nisso ou é melhor usar tudo como CustomVariable mesmo por enquanto?

leomurta commented 3 years ago

Eu achei legal ter os templates de e-mail dessa forma que você falou. Fica mais fácil de customizarmos depois, se for necessário.

braganholo commented 3 years ago

Eu tb gostei da ideia!

JoaoFelipe commented 3 years ago

Feito. Na tela de configurações é possível adicionar templates para emails hardcoded do SAPOS:

image

A parte de usar esse modelo de template para notificações ficou para #360.

Durante a homologação é bom dar uma passada por todas as ações (não é necessário criar os templates) que geram esses emails, já que a forma de referenciar variaveis mudou em relação à forma hardcoded de antes.

As ações são:

Acho que as os emails de trocar senha (devise:password_change) e trocar email (devise:email_changed) não são usados atualmente.

leomurta commented 3 years ago

Obrigado @JoaoFelipe . Já lancei a feature. Por favor, pegue a info que vc detalhou aqui e coloque no manual. Obrigado.