Review Request 109556: Remove KMessageWidget from ContactInfoDialog
David Edmundson
david at davidedmundson.co.uk
Mon Mar 18 01:47:31 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109556/
-----------------------------------------------------------
Review request for Telepathy.
Description
-------
Remove KMessageWidget from ContactInfoDialog
It is perfectly valid for upgradeContact() to fail if FeatureInfo is missing
Also we should NEVER show Tp::PendingOperation error messages to the user
Additional comment, this file should be in widgets, should I move it?
Diffs
-----
KTp/contact-info-dialog.cpp 17ad58390ec356beb04925d2899c3be83b8cf038
Diff: http://git.reviewboard.kde.org/r/109556/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130318/dc1aec52/attachment.html>
More information about the KDE-Telepathy
mailing list