launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #14617
Re: [Merge] lp:~adeuring/launchpad/bug-1079116 into lp:launchpad
Review: Approve
I had a concern about the userCanView calls, but I think your logic about check_permission is correct, so this looks good. Thanks for adding that note in the proposal.
--
https://code.launchpad.net/~adeuring/launchpad/bug-1079116/+merge/136991
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
References