Code review comment for lp:~dandrader/geis/lp939679

Revision history for this message
Stephen M. Webb (bregma) wrote :

In the non-filter debug statement (on or around line 432), could you please make the fact that there is no filter on the subscription explicit? This is important debug information and just leaving it out makes it harder to spot during bug analysis without reading the code.

Other than that, this is good.

review: Approve

« Back to merge proposal