Merge lp:~dylanmccall/harvest/bug-685984 into lp:harvest
Proposed by
Dylan McCall
Status: | Merged |
---|---|
Merged at revision: | 284 |
Proposed branch: | lp:~dylanmccall/harvest/bug-685984 |
Merge into: | lp:harvest |
Diff against target: |
51 lines (+12/-6) 1 file modified
harvest/opportunities/views.py (+12/-6) |
To merge this branch: | bzr merge lp:~dylanmccall/harvest/bug-685984 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Westby | Approve | ||
Review via email: mp+43024@code.launchpad.net |
Description of the change
Some little changes to opportunities.views to ignore invalid opportunities. An invalid opportunity is identified as having its valid property set to False, belonging to an inactive opportunity list, or having a last_updated property different from its opportunity list.
While I was at it, I removed some calls to the .distinct() method for our querysets, which are leftovers from earlier fiddling.
To post a comment you must log in.
Looks good.
Thanks,
James