-
Notifications
You must be signed in to change notification settings - Fork 3
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
1865: Custom application created confirmation mail #1872
1865: Custom application created confirmation mail #1872
Conversation
val backendConfig = context.backendConfiguration | ||
val projectConfig = backendConfig.projects.first { it.id == project } | ||
|
||
for (applicationVerification in verificationEntities) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
even we should only have one organization in this case for verein360, in theory there could be more organization that would all receive this e-mail.
@@ -19,7 +19,7 @@ import java.nio.charset.StandardCharsets | |||
|
|||
object Mailer { | |||
private const val DO_NOT_ANSWER_MESSAGE = | |||
"Dies ist eine automatisierte Nachricht. Bitte antworten Sie nicht auf diese Email." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ui/ux proposed to change the text, so i made a general change for all mails
} | ||
p { +"Bei Rückfragen wenden Sie sich bitte direkt an Ihr zuständiges Landratsamt oder die Verwaltung Ihrer kreisfreien Stadt." } | ||
p { +DO_NOT_ANSWER_MESSAGE } | ||
p { +"- ${projectConfig.administrationName}" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Mit freundlichen Grüßen" is missing (was in the Toni's suggestion)
not sure if it must be included
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alright good find. but then we should add this to all mails
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! thx for the quick fix :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, however, I did not receive any email. I retried with a different email address but to no avail. Any ideas?
did you create a proper |
@seluianova i added a |
@steffenkleinle do you see the email in the log? I don't have smtp configured and have only been checking the logs the whole time, assuming it's enough (isn't it?)
@f1sh1918 I double checked, looks good! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested, thanks for the help. I didn't have anything configured
@seluianova @steffenkleinle |
I think it's quite "safe" from that point of view, because sent emails are printed in the log
|
Short description
As a trainer from verein360, i want to receive a confirmation mail for my application i sent for an applicatant.
Background: For verein360 not the applicant him/herself creates application but the trainer for him/her.
So the confirmation mail should not be send to the applicant but to the creator of the application in this case the trainer.
Proposed changes
Side effects
Testing
Resolved issues
Fixes: #1865