Code review comment for lp:~benji/launchpad/bug-649252

Revision history for this message
Leonard Richardson (leonardr) wrote :

"Neither does Verbose Team Person" doesn't make sense anymore--you should change it to something like "Verbose Team Person gets a concise email, even though they belong to a team that gets verbose email."

I also don't understand why you added the two blank lines starting on line 44 of the diff.

This looks good otherwise.

review: Approve

« Back to merge proposal