• 
      

    Fix issues with the notification manager and tests.

    Review Request #8083 — Created March 29, 2016 and submitted — Latest diff uploaded

    Information

    Review Board
    release-3.0.x
    9ec2e5b...

    Reviewers

    The cleanup change to notification manager seems to have been committed in a
    weird state. The "moved" files just ended up adding the new versions without
    deleting the old, which meant the old tests were still running. Additionally,
    the implementation of the notify method was missing a local variable
    assignment that was needed.

    Ran js-tests.