Clean up desktop notifications
Review Request #8072 — Created March 22, 2016 and submitted — Latest diff uploaded
This patch fixes the last remaining issues on
https://reviews.reviewboard.org/r/7660/, as well as porting the bulk of
the change to ES6.
Ran JS tests.
Diff Revision 2 (Latest)
orig
1
2
reviewboard/staticbundles.py |
---|
reviewboard/static/rb/js/pages/views/reviewablePageView.js |
---|
reviewboard/static/rb/js/pages/views/tests/reviewablePageViewTests.js |
---|
reviewboard/static/rb/js/ui/managers/notificationManagerModel.js |
---|
reviewboard/static/rb/js/ui/managers/tests/notificationManagerModelTests.js |
---|
reviewboard/static/rb/js/ui/views/notificationManager.es6.js |
---|
reviewboard/static/rb/js/ui/views/tests/notificationManagerTests.es6.js |
---|