Review: Approve code Looks good overall.
A minor nitpick: I think that line 212 of the diff should be "The requester is the recipient". -- https://code.launchpad.net/~abentley/launchpad/person-merge/+merge/116964 Your team Launchpad code reviewers is subscribed to branch lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp

