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

Add html-message field #88

Merged
merged 3 commits into from
Aug 30, 2022
Merged

Add html-message field #88

merged 3 commits into from
Aug 30, 2022

Conversation

Findeton
Copy link
Contributor

Add description of the ´html_message´ field that allows sending emails with an html body.

@Findeton Findeton added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 29, 2022
@Findeton Findeton requested a review from edulix August 29, 2022 19:01
Copy link
Contributor

@edulix edulix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just added a few comments after proof-reading, other than that.. please ship it!


This is the HTML body template used for sending the authentication codes to
voters. It's only applicable to authentication methods that send authentication
codes to voters through email, such as `email`, or `email-otp`. In theses this
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be "In these, it corresponds"


Note that to enable using this feature, you need to change the configuration of
the `deployment-tool`, by setting the `config.iam.field allow_html_emails` to
true in the `config.yml` file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to true

the `deployment-tool`, by setting the `config.iam.field allow_html_emails` to
true in the `config.yml` file.

As mentioned earlier, this is a template. Each voter will received a taylored
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will "receive"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants