Page MenuHomePhabricator

Move all email rendering to a template
Closed, ResolvedPublic

Description

Thanks to T240234, we now have a cool 'email.html' template. However, most email rendering is still done in a confusing and cramped way within 'app.py'.

It would be much nicer and easier to work with if we did all rendering in this 'email.html' template.

Event Timeline

Change 559078 had a related patch set uploaded (by Arcayn; owner: Arcayn):
[labs/tools/watch-translations@master] Move all email rendering into a template

https://gerrit.wikimedia.org/r/559078

Arcayn triaged this task as Lowest priority.Dec 18 2019, 8:25 PM

Removing task assignee due to inactivity, as this open task has been assigned for more than two years. See the email sent to the task assignee on February 06th 2022 (and T295729).

Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome.

If this task has been resolved in the meantime, or should not be worked on ("declined"), please update its task status via "Add Action… 🡒 Change Status".

Also see https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator.

Change 921601 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[labs/tools/watch-translations@master] Move all email rendering to a template

https://gerrit.wikimedia.org/r/921601

Change 559078 abandoned by Urbanecm:

[labs/tools/watch-translations@master] Move all email rendering into a template

Reason:

replaced with r921601

https://gerrit.wikimedia.org/r/559078

Change 921601 merged by jenkins-bot:

[labs/tools/watch-translations@master] Move all email rendering to a template

https://gerrit.wikimedia.org/r/921601

Urbanecm claimed this task.