On Fri, 2016-11-11 at 14:33 +0100, Cyril Brulebois wrote: > Is the code really deployed? Yes. > I suspected maybe only new news pages would have this change (others > might have been generated/cached already) but that doesn't seem to be > the case, e.g.: > https://tracker.debian.org/news/814326 If you look in _process_package_event, you will see that it stores HTML in the database! So your change would only affect new events indeed. It looks like what you patched is only used for the plain text news renderer but not for the email news renderer however. > Or maybe I didn't patch the right code path? That looks like the case indeed. These appears to be the right ones: distro_tracker/core/templates/core/news-email.html: email-news-body distro_tracker/core/models.py: EmailNewsRenderer -- bye, pabs https://wiki.debian.org/PaulWise
Attachment:
signature.asc
Description: This is a digitally signed message part