Code review comment for lp:~gmb/launchpad/launchpadformify-subscribe-view-bug-651240

Revision history for this message
Graham Binns (gmb) wrote :

<gmb> allenap: Thanks.
 allenap: So _subscribers_for_current_user is used quite a bit in the subsequent branch, so I'm happier to leave that as-is.
 allenap: And _isSubscriptionRequest() is vestigial and no longer in use.
<allenap> gmb: Yeah, those comments of mine need a lot of salt today.
<gmb> (Which I just noticed)
 allenap: Fair enough. I'll remove one and leave the other, then.

« Back to merge proposal