cjwatsonwgrant: https://code.launchpad.net/~cjwatson/launchpad/team-mail/+merge/269382 could use another look; as well as the obvious changes to fix your review comments, I converted it to use async jobs to make sure that there are no security proxy problems with this branch.  (Might be easiest to look at just r17707.)02:48
cjwatsonAnd now to bed.02:49
wgrantcjwatson: Thanks, those X-L-N-Ts make a tonne more sense now.02:49
cjwatsonPersonTransferJob is kind of a weird home for this, but it already had membership status change notifications ...02:50
=== danilos` is now known as danilos
cjwatsonwgrant: Was there a particular reason you preferred ArchiveJob over DistributionJob for package upload mail?  AFAIK neither is really very relevant - the values saved for the job would be PackageUpload.id, PackageUploadStatus, and summary_text11:33
cjwatsonSo it's maybe more modern-feeling but I'm not seeing how it would actually matter, either would be just for expediency11:33
cjwatsonI guess it sort of categorises them, and it's as good as anything.13:51
wgrantcjwatson: I simply have a great distaste for anything Soyuz-related talking about distributions when the thing is in fact about an archive.23:09
wgrantSee silly things like Distribution.getPublishedSources23:09

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!