Review Request: Initial implementation of telepathy-approver
George Kiagiadakis
kiagiadakis.george at gmail.com
Thu Dec 9 21:27:52 CET 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100199/
-----------------------------------------------------------
Review request for Telepathy.
Summary
-------
This is the initial implementation of the approver. It can approve p2p text chats only, for now. It is build as a kded module. When a new message arrives, it shows a knotify popup, it plays a sound and shows a KStatusNotifierItem flashing in the tray.
gitweb url:
http://gitweb.kde.org/clones/telepathy-approver/gkiagia/telepathy-approver.git/shortlog/refs/heads/implementation
Diffs
-----
CMakeLists.txt PRE-CREATION
src/CMakeLists.txt PRE-CREATION
src/approverdaemon.h PRE-CREATION
src/approverdaemon.cpp PRE-CREATION
src/channelapprover.h PRE-CREATION
src/channelapprover.cpp PRE-CREATION
src/dispatchoperation.h PRE-CREATION
src/dispatchoperation.cpp PRE-CREATION
src/handlewithcaller.h PRE-CREATION
src/handlewithcaller.cpp PRE-CREATION
src/telepathy_kde_approver.desktop PRE-CREATION
src/telepathy_kde_approver.notifyrc PRE-CREATION
src/textchannelapprover.h PRE-CREATION
src/textchannelapprover.cpp PRE-CREATION
src/tpkdeapprovermodule.cpp PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/100199/diff
Testing
-------
Tested with empathy as the chat handler. Currently the "respond" button on the popup doesn't seem to work, but I blame knotify's crappiness for that. Clicking on the flashing icon works perfectly, though.
Thanks,
George
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20101209/212c5ee9/attachment.htm
More information about the KDE-Telepathy
mailing list