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.

Sign up for GitHub

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

Jump to bottom

Add ability to pass an html template when rendering email templates #103

Closed
javialon26 opened this issue Sep 14, 2022 · 2 comments · Fixed by #125
Closed

Add ability to pass an html template when rendering email templates #103

javialon26 opened this issue Sep 14, 2022 · 2 comments · Fixed by #125

Comments

@javialon26
Copy link

It would be really useful to be able to pass an extra template for an HTML message alternative, now only we can send text plain messages. Maybe another configuration OTP_EMAIL_HTML_BODY_TEMPLATE_PATH could be the solution.
Thanks
eugapx pushed a commit to djangoflow/django-otp that referenced this issue Jan 2, 2023
@eugen1j
Add email body_html template (django-otp#103)
c3fa634
eugapx added a commit to djangoflow/django-otp that referenced this issue Jan 2, 2023
@eugapx
Add email body_html template (django-otp#103)
cfc4d85
eugapx added a commit to djangoflow/django-otp that referenced this issue Jan 2, 2023
@eugapx
Add email body_html template (django-otp#103)
6fcb280
@some1ataplace
Copy link

#112 Still needs unit tests

@eschnitzler eschnitzler linked a pull request Jun 15, 2023 that will close this issue
Add email body_html template with tests (#112) #125
Merged
@eschnitzler
Copy link
Contributor

Made a PR that has tests. Please take a look #125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add email body_html template with tests (#112) eschnitzler/django-otp
3 participants
@javialon26 @eschnitzler @some1ataplace