Review Request 121026: Parley: Finally make the "due words" feature work.

Jeremy Whiting jpwhiting at kde.org
Fri Nov 7 00:54:36 UTC 2014


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


Looks good except for the minor things I found. I haven't checked out and built it yet, but if you rebase it on master I can try it quickly (or I can try that here in a bit)


src/collection/collection.h
<https://git.reviewboard.kde.org/r/121026/#comment48942>

    These are new files, right? probably don't need/want Frederik in the copyright of files you created.



src/collection/collection.h
<https://git.reviewboard.kde.org/r/121026/#comment48941>

    No need for a new commented out include.



src/collection/entryfilter.h
<https://git.reviewboard.kde.org/r/121026/#comment48943>

    Should probably update the apidocs when adding a parameter.


- Jeremy Whiting


On Nov. 6, 2014, 5:34 p.m., Inge Wallin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121026/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2014, 5:34 p.m.)
> 
> 
> Review request for KDE Edu and Jeremy Whiting.
> 
> 
> Repository: parley
> 
> 
> Description
> -------
> 
> This patch fixes the so far non-working "due words" feature, which lets the user know how many words are due to practice in each collection in the dashboard.
> 
> 
> Diffs
> -----
> 
>   TODO 3a694f5 
>   src/CMakeLists.txt 3be2282 
>   src/collection/collection.h PRE-CREATION 
>   src/collection/collection.cpp PRE-CREATION 
>   src/collection/entryfilter.h 72cf026 
>   src/collection/entryfilter.cpp 4668e6f 
>   src/collection/testentry.h 572cb26 
>   src/collection/testentry.cpp 04ba8b9 
>   src/dashboard/barwidget.h bcefa6f 
>   src/dashboard/barwidget.cpp 593cd71 
>   src/dashboard/collectionwidget.h 814ece6 
>   src/dashboard/collectionwidget.cpp 573e928 
>   src/dashboard/dashboard.h ac16592 
>   src/dashboard/dashboard.cpp 55413fc 
>   src/practice/sessionmanagerbase.cpp b45e591 
> 
> Diff: https://git.reviewboard.kde.org/r/121026/diff/
> 
> 
> Testing
> -------
> 
> As usual, testing on real-world data in my own studies.
> 
> 
> Thanks,
> 
> Inge Wallin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20141107/5e4887b4/attachment-0001.html>


More information about the kde-edu mailing list