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

Criar uma variável reply_to para escolha de um remetente para os emails automáticos #405

Closed Carlos-Eduardo-Cabral-da-Cunha closed 2 months ago

Carlos-Eduardo-Cabral-da-Cunha commented 2 years ago

Quando o SAPOS envia um email automático, caso a pessoa responda, esse email se perde. Seria desejável poder configurar um email de _replyto, podendo do ser uma variável na tela de variáveis

leomurta commented 1 year ago

@JoaoFelipe , imagino que isso seja fácil de fazer. Depois que finalizar a migração das libs, poderia dar uma olhada nisso? Os e-mails não estão mais se perdendo, mas todos chegam para mim, mesmo os que são de outros programas. Talvez, o ideal seria o reply_to ser o e-mail do admin da instância do Sapos.

nataliacfUFF commented 1 year ago

Oi, pessoal

Sugiro que encaminhem também com cópia para o e-mail da secretaria do programa, pois geralmente são respostas para problemas acadêmicos que o sapos reportou.

Abs,

Natalia

Em qui, 24 de ago de 2023 07:48, Leonardo Gresta Paulino Murta < @.***> escreveu:

@JoaoFelipe https://github.com/JoaoFelipe , imagino que isso seja fácil de fazer. Depois que finalizar a migração das libs, poderia dar uma olhada nisso? Os e-mails não estão mais se perdendo, mas todos chegam para mim, mesmo os que são de outros programas. Talvez, o ideal seria o reply_to ser o e-mail do admin da instância do Sapos.

— Reply to this email directly, view it on GitHub https://github.com/gems-uff/sapos/issues/405#issuecomment-1691448207, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU6PTJE7PZPCGZADKC4NTCLXW4WOXANCNFSM5UWQRH5A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

IgorMonardez commented 3 months ago

Bom dia pessoal, Caso o usuário responda algum e-mail automático do SAPOS, a mensagem seria enviada para esse e-mail reply_to, correto?

JoaoFelipe commented 3 months ago

Isso

IgorMonardez commented 3 months ago

Qual seria o email do admin da instância do SAPOS? sapos@sapos.ic.uff.br?

leomurta commented 3 months ago

Não sei se entendi bem a pergunta, mas vamos lá: como não sabemos como será a instalação do Sapos por cada usuário, é necessário pensarmos numa solução um pouco independente da instância. Na instância do IC, todos os e-mails enviados pelo Sapos saem como sapos@ic.uff.br. Quando alguém responde a esses e-mails (o que pedimos para não ser feito), chega para mim, independentemente da instância (temos 4 em operação nesse momento). Aí eu encaminho para a coordenação do respectivo programa. A ideia é ter um reply-to que usa o e-mail configurado nas variáveis de cada instância para direcionar as respostas. Assim, cada programa decide quem deve receber eventuais respostas a esse e-mail (secretaria, coordenação, etc.)

IgorMonardez commented 3 months ago

Entendi, então cada instância teria um reply_to que seria setado pelo próprio usuário(atravéis das variáveis da configuração), porém se o usuário não setar essa variável haveria algum valor de default para ele ou seguiria com o comportamento padrão do sistema atual?

leomurta commented 3 months ago

Isso. Se o usuário não configurar a variável, fica sem, e segue o comportamento atual. Se derem reply vai para o e-mail que enviou a mensagem (from). Note que em todos os casos o e-mail continua sendo enviado da mesma forma que é hoje. No nosso caso, por sapos@ic.uff.br. Mas teremos a variável "reply-to" configurada no header da mensagem (https://www.gnu.org/software/emacs/manual/html_node/emacs/Mail-Headers.html). Para facilitar, usamos o mesmo nome para a variável do Sapos (reply-to).

leomurta commented 3 months ago

Pessoal, como está o andamento dessa issue? Mandei um e-mail para vcs sugerindo uma mudança no nosso Workflow, para termos mais visibilidade.

IgorMonardez commented 3 months ago

Eu estou ajustando alguns detalhes e tentando emitir a notificação no meu sistema local, como forma de teste

JoaoFelipe commented 3 months ago

O PR parece ok pra mim. @leomurta pode colocar em homologação

(Precisamos fazer alguma coisa para isso? Ex: merge em bugfixes)

leomurta commented 3 months ago

@JoaoFelipe , acho que o ideal é vc ser o gatekeeper, então vc controla a entrada de código no master e no bugfixes, fazendo o merge quando estiver ok. Aí vc me fala qual ramo colocar em homologação. Combinei com a @braganholo de nos ajudar testando.

Se a feature ou correção for secundária, vc pode deixar acumular algumas no ramo antes de me pedir para jogar para homologação. Fica a seu critério.

JoaoFelipe commented 3 months ago

Merge feito. Acho que pode já colocar em homologação, mesmo que não homologue agora: se tiver alguma notificação automatica, já conseguimos pegar para verificar se o reply_to está certo. Também podemos forçar uma notificação, mas dá um pouco mais de trabalho de cadastrar.

leomurta commented 3 months ago

Coloquei em homologação. Testei tanto disparando uma notificação na mão quanto trocando a nota de um aluno. Em ambos os casos os e-mails chegaram e o header de reply-to veio certo. Ao clicar em reply no gmail, ele usou o e-mail do reply-to. Acha que é necessário aguardar pelo timer de notificação? Alguma chance de ter introduzido bug lá? Ou posso colocar em produção?

IgorMonardez commented 3 months ago

Melhor aguardar, apesar de aparentemente eu não ter modificado em nada relacionado a timer de notificação, por não conhecer o sistema muito bem pode ter feito alguma alteração indesejada. Igor Monárdez

Estudante de Ciência da Computação UFF

Em ter., 28 de mai. de 2024 às 14:32, Leonardo Gresta Paulino Murta < @.***> escreveu:

Coloquei em homologação. Testei tanto disparando uma notificação na mão quanto trocando a nota de um aluno. Em ambos os casos os e-mails chegaram e o header de reply-to veio certo. Ao clicar em reply no gmail, ele usou o e-mail do reply-to. Acha que é necessário aguardar pelo timer de notificação? Alguma chance de ter introduzido bug lá? Ou posso colocar em produção?

— Reply to this email directly, view it on GitHub https://github.com/gems-uff/sapos/issues/405#issuecomment-2135776692, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU3E3EUH3ESWNN24VGD7SFDZES5SJAVCNFSM5UWQRH5KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJTGU3TONRWHEZA . You are receiving this because you were assigned.Message ID: @.***>

JoaoFelipe commented 3 months ago

Acho improvável dar erro com o timer, já que funcionou disparando na mão, mas como esse reply_to não é uma necessidade urgente, acho que dá pra esperar (ou cadastrar uma notificação para amanhã e já verificar isso)

leomurta commented 3 months ago

A gente espera. Na virada do mês sempre vem alguma notificação.

leomurta commented 2 months ago

Deu tudo certo na homologação. Passei para produção.