Comment 2 for bug 239864

Revision history for this message
Elliot Murphy (statik) wrote :

I am currently getting email notifications when someone proposes merging to a branch that I own (or is owned by a team that I am a member of). This is great! It looks like I can respond to the merge proposal and my comments get listed as code review comments.

However, I'd like to request two additional bits of information in the notification email that would make the merge proposals even more useful:
1) Include the whiteboard info in the merge proposal notification email. Currently the only content in this email is the urls of the two branches involved in the merge - showing the whiteboard message would allow me to understand why the merge is being requested without having to flip over and load launchpad.

2) Include the diff of the merge, that is the changes that would be made if the merge was approved. This would allow me to do the code review right in my email client, responding to various sections of the diff and then sending the response off to launchpad.