> On Oct. 14, 2013, 12:22 p.m., Martin Klapetek wrote: > > KTp/Declarative/messages-model.h, lines 49-50 > > <http://git.reviewboard.kde.org/r/113222/diff/2/?file=200994#file200994line49> > > > > As discussed on IRC before, this could/should drop the "error" and be > > made more generic for eg. the timestamp of the successful delivery. > > Otherwise the roles count would be growing and growing. > > > > Unrelated, would "DeliveryErrorDebugMessageRole" have some user visible > > data? If so, it shouldn't have "debug" in its name.
In this case it's being used to directly pass the message from Telepathy. I'm not sure that is/should be user-visible. It won't be going through i18n like the text-ui does. Maybe that would be a new role - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113222/#review41709 ----------------------------------------------------------- On Oct. 13, 2013, 10:14 p.m., Leon Handreke wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113222/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2013, 10:14 p.m.) > > > Review request for Telepathy. > > > Repository: ktp-common-internals > > > Description > ------- > > Handle message delivery reports and update original message object with the > the new status and possibly an error message. > > Introduce a new internal MessagePrivate class to hold extra properties for > MessagesModel to avoid having to extend KTp::Message. > > > Diffs > ----- > > KTp/Declarative/messages-model.h a0f15653c3644f26dc768fe7ac882b4ac3c91367 > KTp/Declarative/messages-model.cpp 9cf1606555edc7ed9f36970a957a854caf3010a0 > > Diff: http://git.reviewboard.kde.org/r/113222/diff/ > > > Testing > ------- > > Only tested receiving a Tp::DeliveryStatusDelivered. Not sure how to test the > other cases. > > > Thanks, > > Leon Handreke > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
