launchpad-reviewers team mailing list archive

Thread
Date

[Merge] lp:~stevenk/launchpad/preload-subscriptions into lp:launchpad

To: mp+152073@xxxxxxxxxxxxxxxxxx
From: Steve Kowalik <stevenk@xxxxxxxxxx>
Date: Thu, 07 Mar 2013 01:50:29 -0000
In-reply-to: <20130307011920.19337.28947.launchpad@ackee.canonical.com>
Reply-to: mp+152073@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The proposal to merge lp:~stevenk/launchpad/preload-subscriptions into lp:launchpad has been updated.

Description changed to:

Preload archives and their owners, and pre-cache permissions for Person:+archivesubscriptions.

It turns out the bulk of the queries were trying to work out if the PPA references could be shown. Since subscribers are able to see Archive:+index I dropped the permission required to SubscriberView, and made the code make use of check_permission.

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/preload-subscriptions/+merge/152073
-- 
https://code.launchpad.net/~stevenk/launchpad/preload-subscriptions/+merge/152073
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/preload-subscriptions into lp:launchpad.

References

[Merge] lp:~stevenk/launchpad/preload-subscriptions into lp:launchpad
From: Steve Kowalik, 2013-03-07