Code review comment for lp:~henninge/launchpad/bug-753387-ajax-bug-subscription

Revision history for this message
Данило Шеган (danilo) wrote :

Looks good. As discussed on IRC, I'd only move the mute check inside update_mute_after_subscription_change, but you did say you considered it. So, your call if you want to do that.

review: Approve

« Back to merge proposal