Launchpad itself

Code review comment for lp:~cprov/launchpad/bug-409208-ppa-search-permissions

  1. bug-409208-ppa-search-permissions
  2. Merge into devel
Revision history for this message
Michael Nelson (michael.nelson) wrote :

Great work Celso! The doc test changes document the nuance very well.

My only thought was whether it is worth a comment above the switchDbUser just letting the reader know why it is necessary (for permission on TeamParticipation?).

Thanks!

review: Approve

« Back to merge proposal