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

Allow adding custom HTML to verification emails #582

Open
jonas2515 opened this issue Oct 16, 2023 · 3 comments
Open

Allow adding custom HTML to verification emails #582

jonas2515 opened this issue Oct 16, 2023 · 3 comments

Comments

@jonas2515
Copy link

For compliance with data protection laws in certain situations, it can be necessary to have the full privacy statement of the service as a part of the registration email.

It's already possible to pass a custom body in the verification email using the "Verification email instructions" setting, but that is only made for small strings without any formatting.

Since privacy statements can be quite long, it would be great to be able to use proper formatting for that and pass some HTML to add to the email. Maybe the "Verification email instructions" field could be changed to allow HTML, or maybe a separate field for a custom footer could be added?

Would this be in scope for the app? If yes, I'd be happy to work on a patch for this.

@jonas2515 jonas2515 changed the title Allow adding a custom HTML body to registration emails Allow adding custom HTML to verification emails Oct 16, 2023
@tcitworld
Copy link
Member

Allowing HTML is issue prone (and is tricky right now until nextcloud/server#38093 is in), would markdown support be enough?

(in the meantime you can use the following app to make sure people are aware of your ToS and Privacy Notice : https://apps.nextcloud.com/apps/terms_of_service )

@jonas2515
Copy link
Author

jonas2515 commented Oct 19, 2023

Thanks, nextcloud/server#38093 looks quite interesting, and I definitely see why it's tricky, especially when the HTML comes directly from the user (then again in this case it comes from the admin, so one could argue in this case it doesn't necessarily need sanitizing).

Markdown would work if the resulting text would be left- or right-aligned instead of centered. On the one hand using markdown feels a bit weird given that we're on the web and we're putting together a HTML email here, but on the other hand it would make a ton of sense for generating plaintext emails (could just use the markdown-text directly) and also to support custom style templates for emails. So yeah, on first thought markdown sounds like a more sensible approach to me.

@NeverLoseHope777
Copy link

We would also like to see an HTML or Markdown option. Is this already implemented and I am overlooking it or what are the plans here?

Merci :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants