soyuz emails lack X-LP-Message-Rationale
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Low
|
Colin Watson |
Bug Description
I'm a member of ubuntu-
I just received two emails from soyuz which AFAICT were sent because in each case I approved a sync request (in my capacity as a sponsor) and therefore my name was in the Changed-By field.
This was quite confusing. Note that sponsors are probably doing other things too and may not remember all of the relevant package names from a few days ago.
It would be nice if
* There was an X-LP-M-R header
* It was clear that the Changed-By was synthetic in that all I did was request a sync
Related branches
- William Grant (community): Approve (code)
-
Diff: 3299 lines (+1182/-1013)22 files modifiedlib/lp/archiveuploader/tests/nascentupload-announcements.txt (+54/-53)
lib/lp/archiveuploader/tests/test_ppauploadprocessor.py (+62/-72)
lib/lp/archiveuploader/tests/test_sync_notification.py (+5/-5)
lib/lp/archiveuploader/tests/test_uploadprocessor.py (+111/-98)
lib/lp/services/mail/basemailer.py (+6/-1)
lib/lp/services/mail/notificationrecipientset.py (+29/-5)
lib/lp/soyuz/doc/distroseriesqueue-notify.txt (+99/-56)
lib/lp/soyuz/doc/soyuz-set-of-uploads.txt (+15/-14)
lib/lp/soyuz/emailtemplates/ppa-upload-accepted.txt (+0/-4)
lib/lp/soyuz/emailtemplates/ppa-upload-rejection.txt (+0/-4)
lib/lp/soyuz/emailtemplates/upload-accepted.txt (+0/-4)
lib/lp/soyuz/emailtemplates/upload-new.txt (+0/-4)
lib/lp/soyuz/emailtemplates/upload-rejection.txt (+0/-4)
lib/lp/soyuz/mail/packageupload.py (+533/-535)
lib/lp/soyuz/mail/tests/test_packageupload.py (+205/-100)
lib/lp/soyuz/model/queue.py (+9/-6)
lib/lp/soyuz/scripts/packagecopier.py (+13/-10)
lib/lp/soyuz/scripts/tests/test_copypackage.py (+9/-17)
lib/lp/soyuz/stories/soyuz/xx-queue-pages.txt (+10/-6)
lib/lp/soyuz/tests/test_distroseriesqueue_debian_installer.py (+9/-3)
lib/lp/soyuz/tests/test_packagecopyjob.py (+7/-5)
lib/lp/soyuz/tests/test_packageupload.py (+6/-7)
Changed in soyuz: | |
status: | Confirmed → Triaged |
Changed in launchpad: | |
assignee: | nobody → Colin Watson (cjwatson) |
status: | Triaged → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
Could you attach one of these e-mail messages to the bug report?