Please put the URL to the merge proposal in the body of the email
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Medium
|
Aaron Bentley |
Bug Description
Hi,
Some people will get a merge proposal email without being
intimately familiar with the system. Currently the email isn't very instructive
about how to deal with it, it's just descriptive of the situation.
Putting instructions in the email may not be wise, and we have a web UI
that is designed to lead people through the process, so we should send
inexperienced people there, and let the experienced users do as they like.
Currently the only link in the mail is in the signature, which some mail clients
hide, and isn't front and center so may be missed.
I would suggest putting the link somewhere near the top of the mail, probably
under the requested reviews:
Subject: [Merge] lp:~geser/ubuntu/lucid/gnupg/gnupg-merge-1.4.10-2 into lp:ubuntu/gnupg
Michael Bienia has proposed merging lp:~geser/ubuntu/lucid/gnupg/gnupg-merge-1.4.10-2 into lp:ubuntu/gnupg.
Requested reviews:
Ubuntu Sponsors for main (ubuntu-
See more details and review the request at
https:/
gnupg (1.4.10-2ubuntu1) lucid; urgency=low
* Merge from Debian testing. Remaining changes:
- Add 'debian/
since it fails with ulimit 0 (on buildds).
- Add 'debian/
(or gpg2) and gpgsm to use a passphrase agent by default (lp: 15485)
- Fix udeb build failure on powerpc, building with -O2 instead of -Os.
-- Michael Bienia <email address hidden> Wed, 30 Dec 2009 23:23:09 +0100The attached diff has been truncated due to its size.
[2-line signature. Click/Enter to show.]
--
https:/
Thanks,
James
Related branches
- Brad Crittenden (community): Approve (code)
-
Diff: 272 lines (+79/-40)5 files modifiedlib/canonical/launchpad/emailtemplates/branch-merge-proposal-created.txt (+3/-1)
lib/canonical/launchpad/emailtemplates/branch-merge-proposal-updated.txt (+3/-0)
lib/canonical/launchpad/emailtemplates/review-requested.txt (+3/-0)
lib/lp/code/doc/branch-merge-proposal-notifications.txt (+11/-4)
lib/lp/code/mail/tests/test_branchmergeproposal.py (+59/-35)
tags: | added: bugjam2010 |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
Changed in launchpad: | |
milestone: | none → 11.01 |
Nice suggestion and example.