Review Request: Create plugin to access nepomuk models via QML

[George Goldberg]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102081/#review5079
-----------------------------------------------------------


I'll have some more minor comments later, but first:


src/ui/person-set-model-declarative-wrapper.h
<http://git.reviewboard.kde.org/r/102081/#comment4538>

    Please can you rename this file and class to be EveryonePersonSetModel, since that reflects more accurately what it actually is. 
    
    Also, correct the copyright. Did you copy-paste it from code that was (C) Collabora? If so leave Collabora there, but if you made any modifications that were more than just trivial, add yourself as a copyright holder too.


- George


On July 25, 2011, 10:34 a.m., Francesco Nwokeka wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102081/
> -----------------------------------------------------------
> 
> (Updated July 25, 2011, 10:34 a.m.)
> 
> 
> Review request for Telepathy, George Goldberg and David Edmundson.
> 
> 
> Summary
> -------
> 
> A patch to enable reading info from the nepomuk model via QML
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 0a58700 
>   src/CMakeLists.txt b0cf53b 
>   src/declarative/CMakeLists.txt PRE-CREATION 
>   src/declarative/declarativeplugins.h PRE-CREATION 
>   src/declarative/declarativeplugins.cpp PRE-CREATION 
>   src/declarative/qmldir PRE-CREATION 
>   src/ui/person-set-model-declarative-wrapper.h PRE-CREATION 
>   src/ui/person-set-model-declarative-wrapper.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102081/diff
> 
> 
> Testing
> -------
> 
> This patch works with basic contact list plasmoid in the works in my scratch repo.
> 
> 
> Thanks,
> 
> Francesco
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110725/2ef1e2e4/attachment.htm