- Issue created by @Anybody
- last update
over 1 year ago 1 fail - @anybody opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 5:03pm 26 July 2023 - Assigned to Grevil
- 🇩🇪Germany Anybody Porta Westfalica
@Grevil: This somehow broke tests, but I don't really understand why. Let's have a short chat about this.
What I did was improving the texts, adding
\ninstead of
and re-ordered the config values like in the form. Nothing removed or added there.Any ideas? The tests use their own texts, so this seems strange?
- Status changed to Needs work
over 1 year ago 7:05am 27 July 2023 - 🇩🇪Germany Grevil
You deleted the default events array entry:
- events: ['create'] + events: []Therefore, no mails are sent inside the creation tests. I'll adjust the tests.
- last update
over 1 year ago 11 pass - Status changed to Needs review
over 1 year ago 7:27am 27 July 2023 - 🇩🇪Germany Grevil
All tests are passing! Since I only set the old install config in the tests, I'll RTBC this myself!
RTBC!
- Status changed to RTBC
over 1 year ago 7:35am 27 July 2023 -
Grevil →
committed 8f3a8051 on 2.x authored by
Anybody →
Issue #3377192: Use \n instead of <br> in email templates and improve...
-
Grevil →
committed 8f3a8051 on 2.x authored by
Anybody →
- Status changed to Fixed
over 1 year ago 8:56am 27 July 2023 - Issue was unassigned.
Automatically closed - issue fixed for 2 weeks with no activity.