----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126460/#review89855 -----------------------------------------------------------
This does seem like a much better way to handle the grouping, so I'll +1 this. But as for the actual bug that brought you here: Why are we getting two notifications for everything from Plasma in the first place? - David Edmundson On Dec. 21, 2015, 8:24 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126460/ > ----------------------------------------------------------- > > (Updated Dec. 21, 2015, 8:24 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > https://git.reviewboard.kde.org/r/126459/ introduced new > flag that allows to skip the grouping of notifications. > > This patch adapts the notifications server to that change. > > > Diffs > ----- > > dataengines/notifications/notificationsengine.cpp 829ef4e > > Diff: https://git.reviewboard.kde.org/r/126460/diff/ > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel