[Differential] [Request, 96 lines] D1478: [Notifications] Don't group job finished notifications

[broulik (Kai Uwe Broulik)]

broulik created this revision.
broulik added reviewers: Plasma, mck182, Plasma: Design.
broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace.
broulik added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Otherwise you'll end up with a huge notification containing all previously finished jobs.
  
  BUG: 360156
  FIXED-IN: 5.7.0

TEST PLAN
  Timer applet notification still works.
  
  I also cleaned up a bit there as the "appRealName" thing doesn't deserve a dedicated parameter in the function and was never used there anyway; it's neither exposed through DBus nor as part of the notification service and thus is a private impl I can change.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D1478

AFFECTED FILES
  applets/notifications/package/contents/ui/Jobs.qml
  dataengines/notifications/notificationaction.cpp
  dataengines/notifications/notifications.operations
  dataengines/notifications/notificationsengine.cpp
  dataengines/notifications/notificationsengine.h
  shell/osd.cpp
  shell/osd.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, Plasma, mck182, Plasma: Design
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160424/26eef31c/attachment-0001.html>