> On July 13, 2011, 11:08 p.m., David Edmundson wrote: > > Have you merged this now? > > If so, please close this review - if not do so, because you've already > > shipped the contact list patch that uses this!
I actually haven't. What a fail :/ - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101858/#review4719 ----------------------------------------------------------- On July 11, 2011, 7:36 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101858/ > ----------------------------------------------------------- > > (Updated July 11, 2011, 7:36 p.m.) > > > Review request for Telepathy. > > > Summary > ------- > > This simple patch adds a simple dbus adaptor to the dataengine, so it can be > checked if the plasmoid is in use (the dataengine is created only with the > plasmoid, when not needed, it is deleted). Next part will be checking from > contact list for this dbus entry/method. It is meant as a temp workaround for > bug 270675, once the upstream has proper features, we will remove it. > > > This addresses bug 270675. > http://bugs.kde.org/show_bug.cgi?id=270675 > > > Diffs > ----- > > presence.h 9b199c0 > presence.cpp 34fa5ea > > Diff: http://git.reviewboard.kde.org/r/101858/diff > > > Testing > ------- > > Added the plasmoid, dbus interface was created, removed the plasmoid, dbus > interface gets removed (though the path still stays, but the method is not > valid anymore). > > > Thanks, > > Martin > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
