Review Request 123938: Don't add send delivery messages in the messages vector

Aleix Pol Gonzalez aleixpol at kde.org
Mon Jul 20 11:28:29 UTC 2015


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

(Updated July 20, 2015, 1:28 p.m.)


Status
------

This change has been discarded.


Review request for Telepathy.


Repository: ktp-common-internals


Description
-------

Currently, when using Telegram, we keep getting empty lines, this skips them.

OTOH, it _really_ looks like a bug in the CM, as these messages should go through the received messages rather than the sent, nevertheless I'm unsure if the patch should still go in.


Diffs
-----

  KTp/Declarative/messages-model.cpp dc1088c 

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


Testing
-------

The chat plasmoid works fine.


Thanks,

Aleix Pol Gonzalez

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


More information about the KDE-Telepathy mailing list