[Ml-yokadi] task recurrence

Aurélien Gâteau aurelien.gateau at free.fr
Sun May 17 21:46:39 CEST 2009


Sébastien Renard wrote:
> Hello,
> 
> Please find on "recurrence" git branch a first implementation of recurring 
> tasks.

Looks good! I will soon be able to schedule my car visits to the garage :)

A few remarks, though:
- For consistency, please use camelCase naming for Recurrence methods

- It would be nice to have some unit-tests for this feature

- The syntax to specify the weekday in case of weekly recursion could be
made a bit more forgiving: for example allowing full day name and
localized day name instead of only using the first two chars.
I also suggest lowercasing the whole line before parsing it, to ensure
we do not reject a day written with an upper-cased first letter.

- What do you mean with "recurrence table should fill up and a cleanup
process should be defined"?

Aurélien


More information about the Ml-yokadi mailing list