bshah updated this revision to Diff 80391. bshah marked 3 inline comments as done. bshah added a comment.
- kill separate watcher class - introduce AbstractNotificationsModel - introduce WatchedNotificationsModel REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28509?vs=79122&id=80391 BRANCH bshah/notification-watcher REVISION DETAIL https://phabricator.kde.org/D28509 AFFECTED FILES libnotificationmanager/CMakeLists.txt libnotificationmanager/abstractnotificationsmodel.cpp libnotificationmanager/abstractnotificationsmodel.h libnotificationmanager/abstractnotificationsmodel_p.h libnotificationmanager/dbus/org.freedesktop.Notifications.xml libnotificationmanager/dbus/org.kde.notificationmanager.xml libnotificationmanager/declarative/notificationmanagerplugin.cpp libnotificationmanager/notification.cpp libnotificationmanager/notification.h libnotificationmanager/notification_p.h libnotificationmanager/notificationsmodel.cpp libnotificationmanager/notificationsmodel.h libnotificationmanager/server_p.cpp libnotificationmanager/server_p.h libnotificationmanager/watchednotificationsmodel.cpp libnotificationmanager/watchednotificationsmodel.h To: bshah, #plasma, broulik, davidedmundson Cc: nicolasfella, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart