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

Support HTML email templates #885

Merged
merged 14 commits into from
Apr 3, 2020
Merged

Support HTML email templates #885

merged 14 commits into from
Apr 3, 2020

Conversation

salmar
Copy link
Contributor

@salmar salmar commented Apr 3, 2020

This pull request adds support for HTML templates #884. The following is included:

  • Gradle task to convert MJML templates to HTML templates
  • Enables sending HTML emails (with a plain text alternative for accessibility and to decreased spam score)
  • Adds a system / organization / event property to enable or disable HTML emails.
  • Adds the first HTML templates (confirmation, confirmation for organizers and offline expiring reservations)

This commit adds a Gradle task to generate .ms templates from .mjml templates. The main goal is to be able to send responsive HTML emails.
This commit fixes the exceptions caused by additions to the email data created with Map.of (which returns an immutable Map)
This commit adds the support for rendering multipart emails, rendering both plain text  and HTML versions of the email. It also has the first HTML template for the confirmation email for organizers.
This commit removes the overridable attribute from the TemplateResource constructor as it was set to true in all of them (except the deprecated ones). The overridable property is always set to true, but might be removed in the future.
This commit adds the configuration key to enable HTML emails at system, organization and event levels. The default value is true. When disabled, only plain text templates will be rendered, HTML templates will be omitted.
Make the look and feel similar to the public site design
Prior to this commit, only the plain text message was included in the checksum. This commit also takes the HTML message into account, to avoid false positives in the checksum comparison
@cbellone cbellone self-requested a review April 3, 2020 16:37
@cbellone cbellone merged commit 7d4cac1 into alfio-event:master Apr 3, 2020
@cbellone
Copy link
Member

cbellone commented Apr 3, 2020

merged, thanks! 👍

@syjer syjer mentioned this pull request Sep 2, 2020
38 tasks
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.

2 participants