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

[Martin Klapetek]

> On Jan. 9, 2017, 2:34 p.m., Martin Klapetek wrote:
> > 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
> 
> James Smith wrote:
>     The only public-facing change since David's "Ship it!" was a DBus-specific method name change presenceModelChanged() -> propagationChange(). Also included were the Presence  changes which were also "Ship it!" in review #123482 and were, except for the use of KTp::Presence over Tp::Presence, applicable to this fix.

That doesn't matter, unreviewed patch is unreviewed patch (and the diff is certainly bigger). If you want to continue pushing your changes, please step up as the new maintainer of the project.


- Martin


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


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/20170111/103f6fc8/attachment.html>