[Notifications Engine] Never group notifications with URLs
ClosedPublic

Authored by broulik on Jul 23 2018, 8:16 AM.

Details

Summary

We don't merge URLs and it probably doesn't make sense with the way grouping is currently implemented.

BUG: 396741
FIXED-IN: 5.12.7

Test Plan
notify-send foo bar; notify-send foo baz

was grouped into one popup.
Did two Spectacle screenshots in a row, got two separate notifications

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Jul 23 2018, 8:16 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 23 2018, 8:16 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jul 23 2018, 8:16 AM
davidedmundson accepted this revision.Jul 23 2018, 8:32 AM
This revision is now accepted and ready to land.Jul 23 2018, 8:32 AM
This revision was automatically updated to reflect the committed changes.