Review Request 111256: Import the KPeople KTP integration plugins

Martin Klapetek martin.klapetek at gmail.com
Thu Jun 27 08:23:46 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111256/#review35164
-----------------------------------------------------------

Ship it!


Seems good, but rename the files to go with ktp naming scheme (dashes).

- Martin Klapetek


On June 27, 2013, 12:42 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111256/
> -----------------------------------------------------------
> 
> (Updated June 27, 2013, 12:42 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Import the KPeople KTP integration plugins
> 
> There are two plugins, one for the datasource which provides
> IM data in real time.
> 
> The other is an actions plugin which provides actions that can be started on a PersonsData.
> 
> We need to provide KTp integration into libkpeople in a plugin, but we also want to access this
> singleton of accounts/contacts from the other plugin and from the KTp Models. To do this a static library
> ktpkpeople is created and everything links against that.
> 
> KTp Models the actions plugin and the im data source all link against this.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a83069d59fed945d04560662e3a307ef33b28332 
>   KTp/CMakeLists.txt 6b3de56357ccdbf088a3781864e6826e825a5003 
>   KTp/Models/CMakeLists.txt 720a5b02465b825b08d12d7c43342ad1ba69c9d1 
>   KTp/impersonsdatasource.h PRE-CREATION 
>   KTp/impersonsdatasource.cpp PRE-CREATION 
>   kpeople/actionsplugin/CMakeLists.txt PRE-CREATION 
>   kpeople/actionsplugin/kpeople-actions-plugin.h PRE-CREATION 
>   kpeople/actionsplugin/kpeople-actions-plugin.cpp PRE-CREATION 
>   kpeople/actionsplugin/ktp_kpeople_plugin.desktop PRE-CREATION 
>   kpeople/datasourceplugin/CMakeLists.txt PRE-CREATION 
>   kpeople/datasourceplugin/im_persons_data_source_plugin.desktop PRE-CREATION 
>   kpeople/datasourceplugin/impersonsdatasourceplugin.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111256/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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


More information about the KDE-Telepathy mailing list