Comment 4 for bug 348750
Revision history for this message
| Martin Pool (mbp) wrote Re: [Bug 348750] Re: merge proposal mail context is confusing | #4 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
2009/3/30 Jonathan Lange <email address hidden>:
> Agreed, the To: field is broken. What should it be?
>
> I think we're wrong to use the subject provided by the sender without
> modification. We should prepend some information.
I agree.
> A header at the top of the email might be nice but is not necessary to
> fix this bug IMO. What do our UI ppl think?
fwiw it would be consistent with what's done for questions.
> Martin, I hope you've figured out the riddle of the [Merge] prefix
> (hint: bzr send).
My confusion actually came from not realizing that review responses
could themselves specify a subject, which overrides the default
subject of the merge mails. I realize this is consistent with bug
comments, but is it actually useful? Why not just have one text field
per comment?
--
Martin <http://