Review Request 123487: ktp-contact-list status handler multiple account presence changes

James Smith smithjd15 at gmail.com
Fri Dec 2 22:31:46 UTC 2016


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

(Updated Dec. 2, 2016, 3:31 p.m.)


Review request for Telepathy and David Edmundson.


Changes
-------

1) Replace MainWidget::isAnyAccountOnline() and MainWidget::goOffline() with KTp::GlobalPresence members.


Bugs: 181748
    http://bugs.kde.org/show_bug.cgi?id=181748


Repository: ktp-contact-list


Description (updated)
-------

New classes:
AdvancedPresenceDialog, a dialog for advanced presence setting.

New features:
-GlobalPresenceChooser uses StatusHandlerPresence to enable PluginQueue plugins, and quick status message changes if the status handler is available.
-GlobalPresence is implemented in MainWidget.


Diffs (updated)
-----

  CMakeLists.txt db1ec2ead08daf7176d8e1d014a80d5af249b222 
  dialogs/advanced-presence-dialog.h PRE-CREATION 
  dialogs/advanced-presence-dialog.cpp PRE-CREATION 
  global-presence-chooser.h f07b8823077e1558047ce019f13e619ca6b432c3 
  global-presence-chooser.cpp 323217c82a4ba9f139f38a26734e152915147c25 
  main-widget.h 43334ef8442217154f0df198ce601c9b996914d9 
  main-widget.cpp e0982594ac4c05b84db5772722b98b733410752b 

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


Testing
-------

Compile, run.


File Attachments
----------------

Advanced Presence Setting option
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/06/0f7aa5e8-c3a5-4d2f-9549-32531cddd009__advancedpresencedialogoption.png
Advanced Presence Setting dialog
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/17/b9c5741d-89f6-48f7-887c-68ebffb737c4__advancedpresencedialog.png
Advanced Presence Setting dialog editing status message
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/17/6b368cf1-426e-41da-910e-a732463c42c1__advancedpresencedialogstatusmessage.png


Thanks,

James Smith

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20161202/9ffddf60/attachment.html>


More information about the KDE-Telepathy mailing list