846386 - update kb email templates and rendering for l10n

Status: RESOLVED FIXED

(support.mozilla.org :: General, defect, P2)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

+++ This bug was initially created as a clone of Bug #839157 +++

In bug #827649, I updated the email templates for the users and questions apps so that they can be localized. Bug #839157 coves updating all the templates, but I'm going to reduce the scope of that one by half.

This bug will cover updating these templates:

./wiki/templates/wiki/email/ready_for_review.ltxt
./wiki/templates/wiki/email/approved.ltxt
./wiki/templates/wiki/email/reviewed.ltxt
./wiki/templates/wiki/email/ready_for_l10n.ltxt
./wiki/templates/wiki/email/reviewed_contributors.ltxt
./wiki/templates/wiki/email/edited.ltxt

For each of these, the following needs to happen:

1. convert the template from Django to Jinja2
2. fix the code that renders the template to use the functions in sumo.email_utils rather than Django template renderer
3. test vigorously!


Take a look at the rendering code for email templates in the users and questions apps for examples.

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

In a pull request: https://github.com/mozilla/kitsune/pull/1177

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

Key commit landed in https://github.com/mozilla/kitsune/commit/be99b2d1d0eba576a8bd2f3a34b8327b358ad4d7

Comment 3

[Will Kahn-Greene [:willkg] ET needinfo? me]

Oops--it actually landed in https://github.com/mozilla/kitsune/commit/3457aa1f479a29bfd91e331aa0db6d64a079b32d

Comment 4

[Will Kahn-Greene [:willkg] ET needinfo? me]

This is in production!