> On March 30, 2015, 5:11 p.m., David Edmundson wrote: > > You're not using the new enum?
The plugin using it is in ktp, so cannot be part of this review; I'm finishing that one up. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123184/#review78219 ----------------------------------------------------------- On March 30, 2015, 5:03 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123184/ > ----------------------------------------------------------- > > (Updated March 30, 2015, 5:03 p.m.) > > > Review request for Telepathy and Eike Hein. > > > Repository: kpeople > > > Description > ------- > > Some clients need/want to filter actions based on types, this adds the > actions enum with currently known actions. Plugins returning QList<QAction*> > should then use this enum to set the appropriate type on the action (via > setProperty). > > I've put it to PersonData because all code dealing with actions currently > need to include PersonData header and I'm not sure if it makes sense anywhere > else. > > > Diffs > ----- > > src/persondata.h c3f99a9 > src/widgets/actions.h 2931ef8 > > Diff: https://git.reviewboard.kde.org/r/123184/diff/ > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
