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

[Jeremy Whiting]

> On Nov. 6, 2014, 7:54 p.m., Jeremy Whiting wrote:
> > It looks good but takes a bit to rebase, conflicts with what's in master, mostly because of qt4->qt5 and kdelibs -> kf5 changes. Since this will go to master when it's ready we should probably rebase it right then review. I can try to finish rebasing it tomorrow if needed, otherwise I can push what I have if you want to finish it.

Ok, I rebased it all and it builds and runs, but you should definitely take a look at what I changed. I pushed to collection-class-rebased branch. When I run I see "300 words left" on the dashboard, what other ui changes should I be looking for/at?


- Jeremy


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


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/c39d7cfa/attachment-0001.html>