Code review comment for lp:~dylanmccall/harvest/harvest-dylan-m

Revision history for this message
Daniel Holbach (dholbach) wrote :

Can we get PackageListWrapper.*visible_packages* and PackageListWrapper.*hidden_packages* into opportunity/views.py:opportunities_filter()? We could just pass hidden_packages and visible_packages instead of packages_list. Similarly we could do something for PackageWrapper. Also could we probably do some of the expand logic in javascript. What do you think?

« Back to merge proposal