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

1865: Custom application created confirmation mail #1872

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

f1sh1918
Copy link
Contributor

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

  • if the application was send by verein360 and is pre verified send an e mail to the contact person/trainer of the organization.

Side effects

  • none

Testing

Resolved issues

Fixes: #1865

val backendConfig = context.backendConfiguration
val projectConfig = backendConfig.projects.first { it.id == project }

for (applicationVerification in verificationEntities) {
Copy link
Contributor Author

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."
Copy link
Contributor Author

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}" }
Copy link
Contributor

@seluianova seluianova Jan 21, 2025

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

Copy link
Contributor Author

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

Copy link
Contributor

@seluianova seluianova left a 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 :)

Copy link
Member

@steffenkleinle steffenkleinle left a 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?

@f1sh1918
Copy link
Contributor Author

f1sh1918 commented Jan 22, 2025

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 config.local.yml or there may be an updated doc with a local config file in you user folder. Just check the documentation. Let me know if i you need help @steffenkleinle

@f1sh1918
Copy link
Contributor Author

@seluianova i added a finalInformationParagraph to ensure that the text is equal in all mails. Maybe you can just check if the mail is still ok

@seluianova
Copy link
Contributor

seluianova commented Jan 22, 2025

I did not receive any email. I retried with a different email address but to no avail. Any ideas?

@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?)

i added a finalInformationParagraph to ensure that the text is equal in all mails. Maybe you can just check if the mail is still ok

@f1sh1918 I double checked, looks good!

Copy link
Member

@steffenkleinle steffenkleinle left a 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

@f1sh1918
Copy link
Contributor Author

@seluianova @steffenkleinle
You should configure it soon since you should also be able to check if the template breaks or not for any reason

@seluianova
Copy link
Contributor

seluianova commented Jan 27, 2025

You should configure it soon since you should also be able to check if the template breaks or not for any reason

I think it's quite "safe" from that point of view, because sent emails are printed in the log

[JettyServerThreadPool-38] INFO app.ehrenamtskarte.backend.mail.Mailer - Sending Email:
FromName: Ehrenamtskarte-Bayern-Verwaltung
To: <my-email>
Subject: Antrag erfolgreich eingereicht
-----------------------
Sehr geehrte/r Jane Doe,

Ihr Antrag auf die Bayerische Ehrenamtskarte für John Doe wurde erfolgreich eingereicht.

Den aktuellen Status Ihres Antrags können sie jederzeit unter folgendem Link einsehen. Dort haben Sie auch die Möglichkeit, Ihren Antrag bei Bedarf zurückzuziehen:
https://bayern.ehrenamtskarte.app/antrag-einsehen/ohv35qtAhuQZU22Yo_1aZEUr-qjqF59cQdCJpmkrnjBOOzZn3JBY1Lbfr6yrRXsHs-SL2_rhNc51a_6l8k1IVA%3D%3D 

Bei Rückfragen wenden Sie sich bitte direkt an Ihr zuständiges Landratsamt oder die Verwaltung Ihrer kreisfreien Stadt.

Bitte beachten Sie, dass dies eine automatisierte Nachricht ist. Antworten auf diese E-Mail werden nicht gelesen.

Mit freundlichen Grüßen
- Ehrenamtskarte-Bayern-Verwaltung

@f1sh1918 f1sh1918 merged commit ee262ff into main Jan 27, 2025
2 checks passed
@f1sh1918 f1sh1918 deleted the 1865-custom-application-created-confirmation-mail branch January 27, 2025 14:51
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

Successfully merging this pull request may close these issues.

Send custom application created confirmation mail
3 participants