> On June 25, 2013, 9:48 a.m., Aleix Pol Gonzalez wrote: > > KTp/Models/contacts-model.cpp, line 88 > > <http://git.reviewboard.kde.org/r/111228/diff/1/?file=165661#file165661line88> > > > > Don't you have to set the account manager for the ContactListModel? > > Martin Klapetek wrote: > That is what's happening - there's #ifndef > > David Edmundson wrote: > and assuming the question also meant > "do you have to set it for the PersonsModel". No, it has it's own, > because we didn't want normal KPeople users to have to worry about setting > that (which makes sense I think) > > We do need to figure out what we're doing with the account managers. > Having two factories is very inefficient (you double all DBus traffic), > something needs changing, not sure what.
Uh, my bad. Thanks Martin! >.< Maybe it's possible to re-use the one in KPeople from the ktp-contactlist? Either way, please ship it. /me is excited about finally getting to use it! :D - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111228/#review35033 ----------------------------------------------------------- On June 25, 2013, 1:30 a.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111228/ > ----------------------------------------------------------- > > (Updated June 25, 2013, 1:30 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > Switch ContactsModel between legacy mode and KPeople powered > > Include a proxy model that translates between KPeople and KTp roles so that > it can seemlessly support the two > > > -- > > It includes copied ktptranslationproxymodel, but renamed as it doesn't make > sense to call it the KTp translation inside KTp. > > Code is pushed to kpeople branch. > > I want Martin to cherry-pick (w. review) useful commits on the contact-list > from his branch into master till things vaguely work. > > > Diffs > ----- > > CMakeLists.txt a83069d59fed945d04560662e3a307ef33b28332 > KTp/Models/CMakeLists.txt 720a5b02465b825b08d12d7c43342ad1ba69c9d1 > KTp/Models/contacts-model.cpp 9506151f5489d9a3f80f184834da26cb90141f3c > KTp/Models/kpeopletranslationproxy.h PRE-CREATION > KTp/Models/kpeopletranslationproxy.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/111228/diff/ > > > Testing > ------- > > Opened regular ktp-contact-list. > > It was trying to show things from PersonModel. It was a bit broken, > unsurprisingly, as the old delegates don't support Person rows. > > For some reason presence icons and avatars don't appear. This implies the > translation proxy must be broken, as if we have to change the delegates, then > it won't work with the legacy model. Filteringrtin > > Tried running contact list plasmoid, (as that should 'just work') however we > no longer have anything exposing rolenames so it doesn't work. Needs exposing > into some way that can be shared. (moving setRoleNames from ContactsListModel > to ContactsModel should suffice) > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
