-
-
Notifications
You must be signed in to change notification settings - Fork 884
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
Use proper templating for emails #500
Comments
Yup this is a big one I left for way later. It should probably come after #440 , that way the db could read which languages a user can read, and send emails in a preferred language. |
arewewebyet.org lists some l18n related crates. Looks to me like fluent would be best for our use case. Also this doesnt depend on #440 at all, we already have a language setting in the user profile which can be used for emails. Edit: cargo-i18n might be better, because it supports common file formats. |
Right now the HTML for emails is just hardcoded, which is very ugly and also means that translations are not really possible. We should extract this into template files or something along those lines.
Example, you can find others by grepping for
send_email
.Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: