Review Request 121397: Remove NotifyBySound

Albert Astals Cid aacid at kde.org
Mon Dec 8 22:52:43 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121397/
-----------------------------------------------------------

(Updated Dec. 8, 2014, 10:52 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, David Edmundson and Martin Klapetek.


Repository: knotifications


Description
-------

NotifyByAudio implements the "Sound" notification already

Based on comments from https://git.reviewboard.kde.org/r/121356/


Diffs
-----

  src/knotificationmanager.cpp 9f180b4 
  src/notifybysound.h 1efa620 
  src/notifybysound.cpp 09a80dd 

Diff: https://git.reviewboard.kde.org/r/121397/diff/


Testing
-------


Thanks,

Albert Astals Cid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141208/9d40a903/attachment.html>


More information about the Kde-frameworks-devel mailing list