Merge lp://staging/~ronnie.vd.c/loco-team-portal/reduced_venue_page_queries into lp://staging/loco-team-portal

Proposed by Ronnie
Status: Merged
Approved by: Chris Johnston
Approved revision: 510
Merged at revision: 514
Proposed branch: lp://staging/~ronnie.vd.c/loco-team-portal/reduced_venue_page_queries
Merge into: lp://staging/loco-team-portal
Diff against target: 127 lines (+47/-47)
2 files modified
loco_directory/templates/venues/venue_list.html (+35/-41)
loco_directory/venues/views.py (+12/-6)
To merge this branch: bzr merge lp://staging/~ronnie.vd.c/loco-team-portal/reduced_venue_page_queries
Reviewer Review Type Date Requested Status
Chris Johnston Approve
Review via email: mp+82058@code.staging.launchpad.net

Commit message

Reduced the amount of queries from 800+ to 10 on the event venue list page

Description of the change

Reduced the amount of queries from 800+ to 10 on the event venue list page

Viewing the amount of queries can be done with this example:
http://stackoverflow.com/questions/1074212/show-the-sql-django-is-running/3930290#3930290

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

Works for me

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches