Review Request: Add SSH Contact filtering
David Edmundson
kde at davidedmundson.co.uk
Tue Mar 13 23:51:06 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104265/#review11378
-----------------------------------------------------------
Ship it!
Fine to ship this, but: This needs to change to be more generic.
KTp/Models/accounts-model.h
<http://git.reviewboard.kde.org/r/104265/#comment9105>
Long term we can't make a new role for every new stream tube.
It doesn't make sense to recompile the common library each time someone adds Telepathy support to an app.
KTp/Models/contact-model-item.cpp
<http://git.reviewboard.kde.org/r/104265/#comment9104>
Why can't you do it the same way we do audio/video?
if me->capabilities().streamTubes(QLatin1String("x-ssh-contact"));
- David Edmundson
On March 13, 2012, 11:46 p.m., Daniele Elmo Domenichelli wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104265/
> -----------------------------------------------------------
>
> (Updated March 13, 2012, 11:46 p.m.)
>
>
> Review request for Telepathy.
>
>
> Description
> -------
>
> Add "SSH Contact" filtering to the models
>
>
> Diffs
> -----
>
> KTp/Models/accounts-filter-model.h c23b2843b6ff5a431c4cca275005c1cb4a9ad498
> KTp/Models/accounts-filter-model.cpp 5beb201c1a91ef3347f338d7b5301df372833dc5
> KTp/Models/accounts-model.h a67912d9d4a727da3556a628295cf2e50f49da1e
> KTp/Models/accounts-model.cpp 51fca8802d1522f0b3ed57414c749e8ae0255ece
> KTp/Models/contact-model-item.h 4d8ce512de9541c1013aa5cd38859a9bfd45f1f6
> KTp/Models/contact-model-item.cpp 50145688be749d0bbb83d814ce99f4f02ec387a8
>
> Diff: http://git.reviewboard.kde.org/r/104265/diff/
>
>
> Testing
> -------
>
> Works, but at the moment I don't know how to check if ssh-contact is installed
>
>
> Thanks,
>
> Daniele Elmo Domenichelli
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120313/98b82754/attachment.html>
More information about the KDE-Telepathy
mailing list