search

softwerkskammer / Agora

Die Software für die Plattform der Softwerkskammer
Other
55 stars 59 forks source link

E-Mail-Footer um DeltaTree zu danken #1429

Closed UrsMetz closed 3 years ago

UrsMetz commented 3 years ago

Hier mal ein erster Entwurf für #1428:

Fixes #1428

leider commented 3 years ago

Ist so ok, allerdings fände ich es cool, das erst in dem Moment scharf zu schalten, wenn wir gewechselt haben. Bis dahin könntest Du einen Schalter einbauen in den settings.

leider commented 3 years ago

Oh! - Einer fällt mir noch ein: i18n :)

UrsMetz commented 3 years ago

Ist so ok, allerdings fände ich es cool, das erst in dem Moment scharf zu schalten, wenn wir gewechselt haben. Bis dahin könntest Du einen Schalter einbauen in den settings.

Ja, klingt gut, wo genau würde so eine Einstellung hingehlren (ich habe mal kurz durch den Code geschaut, bin da aber nicht direkt fündig geworden).

UrsMetz commented 3 years ago

Oh! - Einer fällt mir noch ein: i18n :)

Ok, hatte ich schon fast befürchtet, dass i18n noch kommt :-)

UrsMetz commented 3 years ago

Zwei Fragen noch zu i18n:

leider commented 3 years ago

.... ertappt ... Hatte ganz vergessen, das i18n bei Mails natürlich nicht zieht.

leider commented 3 years ago

Konfigurieren über ein der Dateien in config. Mach n neues propery in mailsender-config.json. Beispiel zur Verwendung dann analog mailsenderService.js

function sendMail(message, type, callback) {
  mailtransport.sendMail(message, type, conf.get('sender-address'), callback);
}

Den Flag an das pug zum rendern übergeben und auslesen kannst Du Dir in einem (fast) beliebigen index.jsansehen und von dort dann in das zu rendernde .pugspicken.

UrsMetz commented 3 years ago

Habe eine Config-Option hinzugefügt über die man steuern kann, ob der Footer enthalten ist oder nicht. Dabei ist mir aber folgendes aufgefallen: