Insert tags in mail template not processed

terminal42 / contao-notification_center

The most popular notification configuration extension for the Contao Open Source CMS!

63 stars 38 forks source link

Insert tags in mail template not processed #322

Closed de-es closed 5 months ago

de-es commented 5 months ago

Testumgebung:

Contao 4.13.39
NC 2.x (dev-main)
Offical Demo

Steps to reproduce:

Setze zB {{env::url}} in das Standardtemplate für Mails mail_default.html5.
Erstelle eine Notification.
Löse die Notification aus.

Ergebnis:

Der Insert Tag wird in NC2 nicht ersetzt. In NC1.7 dagegen bei gleichartiger Konfiguration schon.

Eine Regression oder ist das im NC2 erstmal so konzeptionell nicht vorgesehen?

Insert Tags im Mailbody bei Plain Text und HTML funktionieren soweit ohne Probleme.

simon-1977 commented 5 months ago

Ich konnte mir so behelfen und zwar direkt in der Benachrichtigung: {{env::url|urlattr}}/{{link_url::*|urlattr}}

Umgebung:

Contao 4.13.39
NC 2.0.2

de-es commented 5 months ago

Wie gesagt, die Insert Tags direkt in der Benachrichtigung funktionieren. Bei mir sind es die im Mail Template, die im Vergleich zum vorherigen NC nicht mehr ersetzt werden.

Use case: Komplexes, tief verschachteltes, Mailclient-optimiertes Template, wo ich den Redakteuren über den TinyMCE aus gutem Grund nur einen kleinen Teil zur Bearbeitung freigebe.

Toflar commented 5 months ago

https://github.com/terminal42/contao-notification_center/commit/ef06dd7794a2db6eb67bf86336daa578f8cac333

de-es commented 5 months ago

@Toflar Dankeschön für den Fix! :heart: