Review Request 126834: ktp-common-internals propagating presence model for status handler integration and global / multiple account presence fixes

[Martin Klapetek]

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



As you've started pushing unreviewed changes (your ship it was for a different version of the patch), you should either step up as the new maintainer of the project and assume full responsibility for your code or find someone to review and approve your code. But please don't push unreviewed code into master branches, this is in the KDE Telepathy policies as well as general KDE guidelines.

https://community.kde.org/KTp/Developers#Workflow

- Martin Klapetek


On Jan. 9, 2017, 2 p.m., James Smith wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126834/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2017, 2 p.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Repository: ktp-common-internals
> 
> 
> Description
> -------
> 
> New features:
> -Simplified API, sends and collects changes over DBus.
> 
> 
> Diffs
> -----
> 
>   KTp/Models/presence-model.h 8f206b880f48640626322269a14956f105482f69 
>   KTp/Models/presence-model.cpp ddc1a7c75f1a452bf3ac2db1aecbd88a5d1ce519 
> 
> Diff: https://git.reviewboard.kde.org/r/126834/diff/
> 
> 
> Testing
> -------
> 
> Compile, run.
> 
> 
> Thanks,
> 
> James Smith
> 
>

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