Code review comment for lp:~danilo/launchpad/bug-772763-remove-unmute-dialog

Revision history for this message
Brad Crittenden (bac) wrote :

Hi Gary/Danilo,

The branch looks good with the exception that if I am muted and there are no other subscribers then "No subscribers" is correctly shown. However when unmuting, the person's name is added to the subscriber list but "No subscribers" is not removed. Looks like that action should happen around line 119 of the diff.

review: Needs Fixing (code)

« Back to merge proposal