Review Request 123184: Add enum for ActionsType (+ fix documentation)

Martin Klapetek martin.klapetek at gmail.com
Mon Mar 30 17:04:31 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123184/
-----------------------------------------------------------

(Updated March 30, 2015, 5:04 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy and Eike Hein.


Changes
-------

Submitted with commit 5014e971aaaceeb044667d27b5081e5c1e8c6ea6 by Martin Klapetek to branch master.


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/widgets/actions.h 2931ef8 

Diff: https://git.reviewboard.kde.org/r/123184/diff/


Testing
-------


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20150330/6fbbf811/attachment.html>


More information about the KDE-Telepathy mailing list