Code review comment for lp:~brian-murray/launchpad/bug-546078

Revision history for this message
Eleanor Berger (intellectronica) wrote :

As we discussed on IRC, this branch now looks very good. We had one concern, over duplicate results because of the UNION of a subscription to a target and its parent target (like Product and ProjectGroup), so you're going to verify that this isn't the case and fix that if necessary. After that you can land.

review: Approve

« Back to merge proposal