Merge lp:~abentley/launchpad/pre-cache-batches into lp:launchpad
Status: | Merged |
---|---|
Merged at revision: | 14352 |
Proposed branch: | lp:~abentley/launchpad/pre-cache-batches |
Merge into: | lp:launchpad |
Prerequisite: | lp:~abentley/launchpad/view-flags |
Diff against target: |
429 lines (+224/-29) 4 files modified
lib/lp/bugs/browser/bugtask.py (+4/-1) lib/lp/bugs/javascript/buglisting.js (+93/-25) lib/lp/bugs/javascript/tests/test_buglisting.js (+121/-3) lib/lp/services/features/flags.py (+6/-0) |
To merge this branch: | bzr merge lp:~abentley/launchpad/pre-cache-batches |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+82716@code.launchpad.net |
Commit message
Pre-fetch next batch of bug listings.
Description of the change
= Summary =
Implement limited pre-fetching of bug listings.
== Proposed fix ==
The "next" link is pre-fetched, but only if pre-fetching is activated.
== Pre-implementation notes ==
Discussed with deryck
== Implementation details ==
bugs.dynamic_
bugs.dynamic_
The first_batch_config next_batch_config and prev_batch_config are extracted from first_batch, next_batch, and last_batch, because initially, all of those would be pre-fetched.
update ensures we do not pre-fetch batches we already have.
Several methods are updated to understand fetch_only, (i.e. don't render).
== Tests ==
bin/test -t test_buglisting
== Demo and Q/A ==
Ensure bugs.dynamic_
= Launchpad lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/canonical
lib/lp/
lib/canonical
lib/lp/
lib/canonical
lib/lp/
lib/lp/
lib/lp/
Hi Aaron,
Thanks for clearing up my confusion on IRC wrt the Y.bind issue.
typo: ajacent -> adjacent
Your tests provide good coverage and are readable. Thanks.
--Brad