hitobito/hitobito_sac_cas

MAIL: Sprachabhängiger Absendername

Closed this issue · 5 comments

codez commented

Neu sollen alle E-mails der Applikation, welche als Absenderadresse die Standard-Absenderadresse verwenden (Settings.email.sender), je nach gewählter Sprache der E-Mail einen übersetzten Name vorgestellt erhalten.

Die Bezeichnung kann im Locale Yaml hinterlegt werden.

Für den SAC wäre das:

  • de: "Schweizer Alpen-Club SAC <noreply@{mail_domain}>"
  • fr: "Club Alpin Suisse CAS <noreply@{mail_domain}>"
  • it: "Club Alpino Svizzero CAS <noreply@{mail_domain}>"
  • en: "Swiss Alpine Club SAC <noreply@{mail_domain}>"

ToDo

  • Specs schreiben
  • Mit angemessener Rolle "durchklicken"
Largo commented

Aufgefallene Schwierigkeiten:

  • ActionMailer::Base.default from: Settings.email.sender sind globals und daher geteilt mit der ganzen Applikation. Kann daher also nicht per Sprache gesetzt werden. => wir müssen pro E-Mail setzen.
  • Settings sind natürlich auch global.
  • Sprache wird bei jedem Request und relativ spät geladen => dort muss angesetzt werden.
  • In der Konfig muss sender wohl beibehalten werden, weil es bis jetzt Standard war und E-Mails vielleicht auch ausgelöst werden wenn Ubersetzungen nicht geladen sind.
  • In der Konfig ist nicht mehr ersichtlich, dass Sender ueberschrieben wird durch Uebersetzungen.
  • Szenario: Potentielle falsche Übersetzung zerstört E-Mail Versand beim taeglichen Transifex update -> Übersetzung muss validiert werden oder E-Mail Adresse aus Übersetzung entfernen.
  • Architekturfrage: Wie soll mit mehrsprachigen Einstellungen umgegangen werden, sodass das beim mehrsprachig machen der Einstellung keine Anpassung notwendig ist?

=> Mein Lösungsvorschlag mit dem kleinsten Impact auf komplexitaet: Wenn Standardname vor Absenden dann => auf den sprachspezifischen Absender umschreiben.

tobiasstern commented

Hallo

Ich habs auf der Integration getestet und habe hier die französische Mail bekommen, aber mit dem Default/deutschen Absender, siehe Bild:
image

Scheint also noch nicht zu funktionieren.

Largo commented

Ah ja. Die Übersetzung muss noch ins Transifex.

Largo commented

Übersetzung ist nun im Transifex.

tobiasstern commented

Funktioniert 🙌