Review Request 114718: Remove ConversationTarget, put as title/avatar/presenceIcon into Conversation

Leon Handreke leonh at ndreke.de
Sun Dec 29 08:25:24 UTC 2013


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

(Updated Dec. 29, 2013, 8:25 a.m.)


Review request for Telepathy.


Changes
-------

Forgot to commit two lines.


Repository: ktp-common-internals


Description
-------

Remove ConversationTarget, put as title/avatar/presenceIcon into Conversation


Diffs (updated)
-----

  KTp/Declarative/CMakeLists.txt 5517c31d511e04a0d695290d5044cf16bc958478 
  KTp/Declarative/conversation-target.h e620cdc75193e78f31e5d6a8cefb3d8b2a87b6cd 
  KTp/Declarative/conversation-target.cpp 4e8af19397ec77fecf1a876d7378119204f02932 
  KTp/Declarative/conversation.h 38b313e700244173e8988e1e65cf48f82e04288b 
  KTp/Declarative/conversation.cpp a3644fd6a2726a760ec4dfc3682cd3e93134f00b 
  KTp/Declarative/conversations-model.cpp 4e3489adc07bc8d724a5456ee371bac32ea4add3 
  KTp/Declarative/pinned-contacts-model.cpp ccb5f2d251fb4c4bd7625d9c1ac527f4d1bc64b1 
  KTp/Declarative/qml-plugins.cpp 88e08bffc75809b3863ac0915c0ef53539c0810f 

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


Testing
-------

Tested plasmoid, still works fine.

Flipped the if in the Conversation constructor to make the Conversation think that it's a group chat, plasmoid still works (but doesn't show proper name, avatar and presence) and crashes when trying to pin contacts (the "star" button).


Thanks,

Leon Handreke

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


More information about the KDE-Telepathy mailing list