Mark "Connection Activated" notifications as low priority
ClosedPublic

Authored by ngraham on Thu, May 9, 6:08 PM.

Details

Summary

In 5.16 onwards, low priority notifications are displayed to the user in a pop-up but
don't get added to the history.

When a network connection is successfully activated, there's nothing further for the
user to have to know or do; it just works and they start using it. As such, there is
no value to keeping the notification in the history; it's just spam. So, this patch
marks that notification as low priority so it doesn't clutter up the history.

Test Plan
  • Apply patch
  • Reboot or switch to a different network connection
  • See "Connection activated" notification
  • Open up notifications history, see nothing there, be happy!

Diff Detail

Repository
R116 Plasma Network Management Applet
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Thu, May 9, 6:08 PM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, May 9, 6:08 PM
Restricted Application added a reviewer: jgrulich. · View Herald Transcript
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Thu, May 9, 6:08 PM
jgrulich accepted this revision.Fri, May 10, 5:53 AM
This revision is now accepted and ready to land.Fri, May 10, 5:53 AM
This revision was automatically updated to reflect the committed changes.