• 
      

    Making call to DnsRequest more explicit.

    Review Request #6663 — Created Dec. 2, 2014 and submitted — Latest diff uploaded

    Information

    Review Board
    master
    0e62407...

    Reviewers

    Update email notification logic to honor should_send_own_update flag

    Fixes: 3684, 3681

    If a user was explicitly listed in a review request along with also being a member of a group assigned to a request, the should_send_own_update setting in the users profile would not be honored because the check was happening before we added explicit users to the to_field list. This also fixes the issue where we were trying to remove email address that might not actually exist (a blank review request).

    Had a user create a review request that lists me as an explicit reviewer along with a group that I'm also a member of; commented on the issue and I never recieved an email notification.