search

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
28 stars 14 forks source link

Issue 405 #463

Closed IgorMonardez closed 6 months ago

IgorMonardez commented 6 months ago

Nessa Issue eu entendi desse jeito, como todo e-mail respondido iria para a adm da sapos, que eu supus ser sapos@sapos.ic.uff.br, todo e-mail respondido pelo usuário iria para ele e teria ainda outra reply_to definida via variável do usuário.

JoaoFelipe commented 6 months ago

O template de email deveria carregar o email padrão de CustomVariables ao invés de usar sapos@sapos.ic.uff.br hardcoded. Além disso, seria bom alterar também o mailer para usar o reply_to

Não cheguei a olhar a fundo a parte de notification_logs que foi alterada. Não sei se tem muita necessidade de armazenar o reply_to, mas não vejo problema em ter.

IgorMonardez commented 6 months ago

Portanto o email @.** entraria como seed do projeto na CustomVariable? Igor Monárdez*

Estudante de Ciência da Computação UFF

Em dom., 19 de mai. de 2024 às 17:06, João Felipe N. Pimentel < @.***> escreveu:

O template de email deveria carregar o email padrão de CustomVariables ao invés de usar @.*** hardcoded. Além disso, seria bom alterar também o mailer para usar o reply_to

Não cheguei a olhar a fundo a parte de notification_logs que foi alterada. Não sei se tem muita necessidade de armazenar o reply_to, mas não vejo problema em ter.

— Reply to this email directly, view it on GitHub https://github.com/gems-uff/sapos/pull/463#issuecomment-2119345402, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU3E3EUFRMXYACEWMF5KZXLZDEA25AVCNFSM6AAAAABH6MA6CCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGM2DKNBQGI . You are receiving this because you authored the thread.Message ID: @.***>

leomurta commented 6 months ago

@IgorMonardez , veja o comentário que eu coloquei agora lá na issue. Acho que vai ficar mais claro como deve ser o encaminhamento.

Quanto a mensagem dos commits, note que no padrão sugerido pelo João, o "if applied, this commit will" não entra na mensagem em si. Só o restante. Releia aqui: https://cbea.ms/git-commit/. Eu tenho o hábito de escrever no singular: "Adds bla bla bla", "Fixes bla bla bla". Os que já fez, deixe como está sem problemas.

IgorMonardez commented 6 months ago

Bom dia, atualizei essa branch com mais um push

IgorMonardez commented 6 months ago

Fiz mais um push relacionado a esse pull request